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

Site preview images and some other metadata bits #22

Merged
merged 4 commits into from
Feb 16, 2021

Conversation

omgmog
Copy link
Member

@omgmog omgmog commented Feb 15, 2021

This should give us working /preview/XYZ.png previews for use on Twitter and other social shares.

These are generated automatically 🎉

home
2021-01
code-of-conduct

This will make tweets like this: https://twitter.com/JSOxford/status/1359148138204966916 look a lot COOLER 😎

@omgmog
Copy link
Member Author

omgmog commented Feb 15, 2021

Screenshot 2021-02-15 at 17 53 45

💥

@omgmog
Copy link
Member Author

omgmog commented Feb 15, 2021

Looks like the meta descriptions could do with an added | safe to handle entity encoding better 🤔

Don't merge this until I've pushed a commit for that

src/_includes/layouts/base.njk Show resolved Hide resolved
src/_includes/layouts/base.njk Show resolved Hide resolved
src/_includes/layouts/base.njk Show resolved Hide resolved
src/_includes/layouts/base.njk Show resolved Hide resolved
@omgmog
Copy link
Member Author

omgmog commented Feb 16, 2021

Oh I did not mean to push that, bare with 💯

@omgmog omgmog requested a review from javorszky February 16, 2021 12:35
@omgmog
Copy link
Member Author

omgmog commented Feb 16, 2021

I think this is ready for a review 👍

@javorszky javorszky merged commit 555ead1 into jsoxford:main Feb 16, 2021
@omgmog omgmog deleted the og-preview-images branch February 16, 2021 13:47
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.

2 participants