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

Open graph tags #95

Merged
merged 2 commits into from Jun 16, 2016
Merged

Open graph tags #95

merged 2 commits into from Jun 16, 2016

Conversation

durden
Copy link
Contributor

@durden durden commented Jun 16, 2016

No description provided.

… on FB

- Also add og:image tag to guide pages so the stack icon shows up if it's
  available.
- Note FB open graph tags require external URLs
…ide pages

- Add explicit width/height and description to satisfy open graph debugger warnings
- This will fallback to the user's avatar image if the stack image isn't the
  right format.
- We're planning on transitioning to png for stack images so this should
  automatically work once that change happens.
    - Follow #89 for more discussion on that topic.
@durden
Copy link
Contributor Author

durden commented Jun 16, 2016

This change adds the open graph meta tags to every page of the site for easier sharing. All pages except the guide pages will use the hack.guides() logo and generic site description. The guide pages will use a description like 'Guide related to ', the stack image if it's png/jpg/jpeg (see #89 for more info), and the guide title.

For reference, this improvement was first suggested by Rishi Athanikar in this tweet.

@durden durden merged commit df2d6db into master Jun 16, 2016
@durden durden deleted the open_graph_tags branch June 16, 2016 07:54
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

1 participant