Skip to content

Commit cdeb4ed

Browse files
authored
Fix bug with ogp_image_alt default (#29)
1 parent 62cda7e commit cdeb4ed

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

sphinxext/opengraph/__init__.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,9 @@ def get_tags(
9090
# Add image alt text (either provided by config or from site_name)
9191
if isinstance(ogp_image_alt, str):
9292
tags += make_tag("og:image:alt", ogp_image_alt)
93-
elif ogp_image_alt and site_name:
93+
elif ogp_image_alt is None and site_name:
9494
tags += make_tag("og:image:alt", site_name)
95-
elif ogp_image_alt and title:
95+
elif ogp_image_alt is None and title:
9696
tags += make_tag("og:image:alt", title)
9797

9898
# custom tags
@@ -116,7 +116,7 @@ def setup(app: Sphinx) -> Dict[str, Any]:
116116
app.add_config_value("ogp_site_url", None, "html")
117117
app.add_config_value("ogp_description_length", DEFAULT_DESCRIPTION_LENGTH, "html")
118118
app.add_config_value("ogp_image", None, "html")
119-
app.add_config_value("ogp_image_alt", True, "html")
119+
app.add_config_value("ogp_image_alt", None, "html")
120120
app.add_config_value("ogp_use_first_image", False, "html")
121121
app.add_config_value("ogp_type", "website", "html")
122122
app.add_config_value("ogp_site_name", None, "html")

0 commit comments

Comments
 (0)