-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add opengraph meta tags #3247
Add opengraph meta tags #3247
Conversation
edadd43
to
4e02f5a
Compare
31f964b
to
0eb49c4
Compare
docs/templates/shared/_head.html.j2
Outdated
<meta property="og:description" content="jq is a lightweight and flexible command-line JSON processor"> | ||
<meta property="og:type" content="website"> | ||
<meta property="og:image" content="{{ url }}{{ root }}/icon.png"> | ||
<meta property="og:url" content="{{ url }}{{ root }}/{{ permalink }}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah this should probably not have root, fixing
Makes the site preview a bit nicer when sharing
Should we just remove |
Never mind, now they are the same :) |
👍 yeah i noticed my own confusion hehe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Makes the site preview a bit nicer when sharing