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

Icon does not scale well on Ubuntu #391

Closed
kevinhikaruevans opened this issue Dec 16, 2020 · 2 comments · Fixed by #395
Closed

Icon does not scale well on Ubuntu #391

kevinhikaruevans opened this issue Dec 16, 2020 · 2 comments · Fixed by #395
Labels
installers/deb Issues related to the Debian installer

Comments

@kevinhikaruevans
Copy link

kevinhikaruevans commented Dec 16, 2020

Describe the feature or problem you’d like to solve

The icon does not downscale well on Ubuntu 20.04. I'm using Github Desktop 2.5.4-linux1. Here's a screenshot of my activity bar next to other applications:
image

This is a fairly minor problem but it just looks odd next to the other icons.

Proposed solution

The ideal solution would be to include 128x128, 64x64, 32x32 prescaled icons and an svg. I think the issue is downscaling is done using nearest neighbor on a large icon, and it'd be better to just include the smaller icons instead and a vector image.

Here's what it should look like in comparison:

image

@shiftkey
Copy link
Owner

@kevinhikaruevans thanks for the report. I'm a bit puzzled about this one but here's some context about the packaging side of things.

These are the versions I'm specifying to electron-installer-debian:

icon: {
'256x256': 'app/static/logos/256x256.png',
'512x512': 'app/static/logos/512x512.png',
'1024x1024': 'app/static/logos/1024x1024.png',
},

This seems to work fine based on the docs: https://github.com/electron-userland/electron-installer-debian#optionsicon

I'm not sure if there's an SVG version of the icon around that could be used here, but it would be worth checking the installer is actually using these values, and not falling back to something else unexpected here.

@shiftkey shiftkey added the installers/deb Issues related to the Debian installer label Dec 16, 2020
@kevinhikaruevans
Copy link
Author

I've added a PR that adds in the smaller icons. I tested it and it looks less jagged. 👍

There is an svg available here: https://desktop.github.com/images/desktop-icon.svg but it looks a little different from the pngs, so I didn't include it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
installers/deb Issues related to the Debian installer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants