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

Compress discover page images #2038

Merged
merged 3 commits into from
Feb 1, 2023
Merged

Compress discover page images #2038

merged 3 commits into from
Feb 1, 2023

Conversation

jankoegel
Copy link
Contributor

@jankoegel jankoegel commented Feb 1, 2023

⚠️ do not merge until the files have been uploaded to the DigitalOcean buckets (see TODOs below)

What

  • adds Hamstr to list of Nostr clients.

  • reduces total file size from 2.4MB to 179kB.

  • Compressed images will be uploaded to DigitalOcean (DO) bucket. File extension were changed from .svg to .png because those were either:
    a. fake SVGs that only contained embedded PNGs (I extracted the PNG content, so it's a real PNG now - no worries).
    b. real SVG files I found on the net did not fit square formatting or had weird viewbox/size parameters so I couldn't get them to fit into our page without editing the SVGs in Adobe Illustrator -> converted to PNG with photoshop

  • PNGs were scaled to 256x256px where the original PNG was large enough. One original image was slightly smaller and I kept its original size. 256px is more than enough for the 56x56 CSS-pixel elements we have on the discover page.

  • 📥 DOWNLOAD compressed-icons.zip

TODOs

Screenshots

chrome-extension___ilikpdojllekoampbknappljinadggld_options html

#2018

Compressed images will be uploaded to DigitalOcean (DO) bucket.
changed file extension from SVG to PNG because those were:
a. fake SVGs that only contained embedded PNGs
b. real SVG files I found on the net did not fit square formatting or
had weird viewbox/size parameters so I couldn't get them to fit into
our page without editing the SVGs in Adobe Illustrator.

+ PNGs are scaled to 256x256 (where the original PNG was large enough,
one was slightly smaller and I kept its original size), which is more
than enough for the 56x56 CSS pixel elements we have on the discover
page.
@@ -5,19 +5,19 @@
{
"title": "LNMarkets",
"subtitle": "Instant Bitcoin derivatives trading",
"logo": "https://cdn.getalby-assets.com/alby-extension-website-screen/lnmarketes.svg",
"logo": "https://cdn.getalby-assets.com/alby-extension-website-screen/lnmarkets.png",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed typo in file name

This should be the same bucket under a different name / domain.
@github-actions
Copy link

github-actions bot commented Feb 1, 2023

🚀 Thanks for the pull request!

Here are the current build files for testing:

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.


This build is brought to you by: channel.ninja (who recently dropped 1000 sats):


Want to sponsor the next build? send some sats to ⚡️builds@getalby.com (don't forget to provide your name)

Don't forget: keep earning sats!

Nostr clients: add Hamstr, ABC-sort.
@jankoegel jankoegel marked this pull request as ready for review February 1, 2023 21:06
@bumi bumi merged commit 0e0ff20 into master Feb 1, 2023
@bumi bumi deleted the chore/compress-discover-images branch February 1, 2023 21:11
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.

Better compress images on the discover screen
2 participants