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

Meta tags: Change opengraph logo and add possiblity to use custom one #1002

Merged
merged 1 commit into from Jun 24, 2020

Conversation

erciccione
Copy link
Contributor

@erciccione erciccione commented May 24, 2020

The old logo showing on social medias is too large and end up being trimmed:

Twitter:
twitter

Reddit:
reddit

Replaced it with the simple Monero Logo. If a logo is specified using 'image:' in the front matter, jekyll will show this one instead. This also allow us to easily make specific meta-images for each blog post. I will add the possibility to add the image in the body of the post in a subsequent PR.

@erciccione erciccione marked this pull request as draft May 24, 2020 11:08
@erciccione erciccione force-pushed the meta-logo branch 2 times, most recently from 21eabef to 91247bf Compare May 24, 2020 11:18
@erciccione erciccione marked this pull request as ready for review May 24, 2020 11:19
@erciccione
Copy link
Contributor Author

erciccione commented Jun 19, 2020

Changed page.logo to page.image. For compatibility with the plugin for the RSS feed i'm currently testing (#916)

@netlify
Copy link

netlify bot commented Jun 19, 2020

Deploy preview for zen-roentgen-73fc0e ready!

Built with commit 18bedd3

https://deploy-preview-1002--zen-roentgen-73fc0e.netlify.app

erciccione added a commit to erciccione/monero-site that referenced this pull request Jun 22, 2020
- Use 'jekyll-feed' plugin to manage RSS feed and remove existing feed.xml file
- Add feed image in main blog tab ('All posts')

The 'jekyll-feed' plugin makes possible for aggregators to automatically detect the existence of a feed (feed_meta tag in header)
and creates a 'feed.atom' file containing the last 20 blog posts and locates it in the root directory. The structure is compatible
with our current system, including the possibility to show a picture if it's included in the post (monero-project#1002).
erciccione added a commit to erciccione/monero-site that referenced this pull request Jun 22, 2020
- Use 'jekyll-feed' plugin to manage RSS feed and remove existing feed.xml file
- Add feed image in main blog tab ('All posts')

The 'jekyll-feed' plugin makes possible for aggregators to automatically detect the existence of a feed (feed_meta tag in header)
and creates a 'feed.atom' file containing the last 20 blog posts and locates it in the root directory. The structure is compatible
with our current system, including the possibility to show a picture if it's included in the post (monero-project#1002).
_includes/head.html Outdated Show resolved Hide resolved
The old logo showing on social medias is too large and end up being trimmed. Replaced it with the simple Monero Logo.
If an image is specified using 'image:' in the front matter of a blog post, jekyll will show that image instead. This also allow us to easily
make specific meta-images for each blog posts. I will add the possibility to add the image in the body of the post in a
subsequent PR.
@luigi1111 luigi1111 merged commit 56ce797 into monero-project:master Jun 24, 2020
@erciccione erciccione deleted the meta-logo branch June 24, 2020 18:52
erciccione added a commit to erciccione/monero-site that referenced this pull request Jul 29, 2020
- Use 'jekyll-feed' plugin to manage RSS feed and remove existing feed.xml file
- Add feed image in main blog tab ('All posts')

The 'jekyll-feed' plugin makes possible for aggregators to automatically detect the existence of a feed (feed_meta tag in header)
and creates a 'feed.atom' file containing the last 20 blog posts and locates it in the root directory. The structure is compatible
with our current system, including the possibility to show a picture if it's included in the post (monero-project#1002).
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

4 participants