don't require the host to be in the Image tag
now that we know our base host via config, we can stop hardcoding it in each Image tag Signed-off-by: Brian S. Stephan <bss@incorporeal.org>
This commit is contained in:
parent
abc05ee4e8
commit
6d7987cfae
@ -90,7 +90,7 @@ def handle_markdown_file_path(path: str) -> str:
|
|||||||
return template.render(title=page_title,
|
return template.render(title=page_title,
|
||||||
config=Config,
|
config=Config,
|
||||||
description=get_meta_str(md, 'description'),
|
description=get_meta_str(md, 'description'),
|
||||||
image=get_meta_str(md, 'image'),
|
image=Config.BASE_HOST + get_meta_str(md, 'image'),
|
||||||
content=content,
|
content=content,
|
||||||
base_url=Config.BASE_HOST + instance_resource_path_to_request_path(path),
|
base_url=Config.BASE_HOST + instance_resource_path_to_request_path(path),
|
||||||
navs=parent_navs,
|
navs=parent_navs,
|
||||||
|
@ -5,9 +5,9 @@ SPDX-License-Identifier: GPL-3.0-only
|
|||||||
<html lang="en">
|
<html lang="en">
|
||||||
<title>{{ title }}</title>
|
<title>{{ title }}</title>
|
||||||
<meta charset="utf-8">
|
<meta charset="utf-8">
|
||||||
|
<meta property="og:url" content="{{ base_url }}">
|
||||||
{% if title %}<meta property="og:title" content="{{ title }}">{% endif %}
|
{% if title %}<meta property="og:title" content="{{ title }}">{% endif %}
|
||||||
{% if description %}<meta property="og:description" content="{{ description }}">{% endif %}
|
{% if description %}<meta property="og:description" content="{{ description }}">{% endif %}
|
||||||
<meta property="og:url" content="{{ base_url }}">
|
|
||||||
{% if image %}<meta property="og:image" content="{{ image }}">{% endif %}
|
{% if image %}<meta property="og:image" content="{{ image }}">{% endif %}
|
||||||
<meta name="twitter:card" content="summary_large_image">
|
<meta name="twitter:card" content="summary_large_image">
|
||||||
<meta name="viewport" content="width=device-width, initial-scale=1">
|
<meta name="viewport" content="width=device-width, initial-scale=1">
|
||||||
|
@ -63,3 +63,16 @@ def test_figures_are_rendered():
|
|||||||
'<span></span></figure>') in data
|
'<span></span></figure>') in data
|
||||||
assert '<figure><img alt="just a logo" src="bss-square-no-bg.png" /></figure>' in data
|
assert '<figure><img alt="just a logo" src="bss-square-no-bg.png" /></figure>' in data
|
||||||
os.chdir(HERE)
|
os.chdir(HERE)
|
||||||
|
|
||||||
|
|
||||||
|
def test_og_image():
|
||||||
|
"""Test that the og:image meta tag is present as expected."""
|
||||||
|
with tempfile.TemporaryDirectory() as tmpdir:
|
||||||
|
src_dir = os.path.join(HERE, 'instance')
|
||||||
|
ssg = StaticSiteGenerator(src_dir, tmpdir)
|
||||||
|
os.chdir(os.path.join(src_dir, 'pages'))
|
||||||
|
|
||||||
|
ssg.build_file_in_destination(os.path.join(HERE, 'instance', 'pages'), '', 'more-metadata.md', tmpdir, True)
|
||||||
|
with open(os.path.join(tmpdir, 'more-metadata.html'), 'r') as graphviz_output:
|
||||||
|
data = graphviz_output.read()
|
||||||
|
assert ('<meta property="og:image" content="http://example.org/test.img') in data
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
Title: title for the page
|
Title: title for the page
|
||||||
Description: description of this page
|
Description: description of this page
|
||||||
made even longer
|
made even longer
|
||||||
Image: http://buh.com/test.img
|
Image: /test.img
|
||||||
|
|
||||||
hello
|
hello
|
||||||
|
Loading…
Reference in New Issue
Block a user