Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable ability to add social markup tags to <head>. #75

Merged
merged 1 commit into from Apr 2, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 10, 2015

h/t to https://github.com/DandyDev/pelican-bootstrap3/

Note I am not an expert on social meta tags (or on this theme / pelican) and am open to feedback / pushback / guidance / etc.

I have implemented this on one post so far. See links for Twitter card and Google+ share.

@@ -25,6 +32,45 @@
{{ EXTRA_HEADER }}
{% endif %}

{% if article %}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it make more sense to move this block of code in a separate template to include here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update on the way.

@dhermes dhermes force-pushed the add-social-sharing-metadata branch from e683922 to bcfa142 Compare April 1, 2015 16:25
@dhermes
Copy link
Contributor Author

dhermes commented Apr 1, 2015

@duilio I rebased and pushed the requested changes.

duilio added a commit that referenced this pull request Apr 2, 2015
Enable ability to add social markup tags to <head>.
@duilio duilio merged commit 29020e0 into duilio:master Apr 2, 2015
@duilio
Copy link
Owner

duilio commented Apr 2, 2015

Nice job ;-)

@dhermes dhermes deleted the add-social-sharing-metadata branch April 2, 2015 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants