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

feat: update favicon and og #2148

Merged
merged 2 commits into from
May 20, 2024
Merged

Conversation

alexiscolin
Copy link
Member

This PR updates favicon and og images / info.

  • More favicon formats and sizes (for quality, compatibility and HD matters)
  • All images have been optimized to reduce their sizes
  • Adds a default og image to improve UX and SEO when users share the website
  • Completes basic og info such as title and description

Note:
Would be cool to use a themed (light/dark) favicon by using native prefered-color-scheme in the next gnoweb iteration (require to update the light/dark theme system a bit)

We could also add a webmanifest.jsonto give more details to android devices (when users install the website as a webapp and/or pin it to their homescreen). Same for IE with a msImage and a browserconfig.xml file

@github-actions github-actions bot added the 📦 ⛰️ gno.land Issues or PRs gno.land package related label May 20, 2024
Copy link

codecov bot commented May 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.03%. Comparing base (c4cffb5) to head (e1bc6be).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2148   +/-   ##
=======================================
  Coverage   49.03%   49.03%           
=======================================
  Files         576      576           
  Lines       77556    77556           
=======================================
  Hits        38032    38032           
  Misses      36438    36438           
  Partials     3086     3086           
Flag Coverage Δ
gno.land 61.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alexiscolin alexiscolin added the don't merge Please don't merge this functionality temporarily label May 20, 2024
@moul moul merged commit 228e9d0 into gnolang:master May 20, 2024
144 checks passed
leohhhn pushed a commit to leohhhn/gno that referenced this pull request May 21, 2024
This PR updates favicon and og images / info.

- More favicon formats and sizes (for quality, compatibility and HD
matters)
- All images have been optimized to reduce their sizes
- Adds a default `og` image to improve UX and SEO when users share the
website
- Completes basic `og` info such as `title` and `description`

Note:
_Would be cool to use a themed (light/dark) favicon by using native
`prefered-color-scheme` in the next `gnoweb` iteration (require to
update the light/dark theme system a bit)_

_We could also add a `webmanifest.json`to give more details to android
devices (when users install the website as a webapp and/or pin it to
their homescreen). Same for IE with a `msImage` and a browserconfig.xml
file_
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
don't merge Please don't merge this functionality temporarily 📦 ⛰️ gno.land Issues or PRs gno.land package related
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants