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

KDE uses generic "desktop" icon rather than GitHubDesktop icon #42

Closed
jkbest2 opened this issue Apr 20, 2018 · 8 comments · Fixed by #80
Closed

KDE uses generic "desktop" icon rather than GitHubDesktop icon #42

jkbest2 opened this issue Apr 20, 2018 · 8 comments · Fixed by #80
Assignees

Comments

@jkbest2
Copy link

jkbest2 commented Apr 20, 2018

Description

Admittedly, this is a very minor issue, and I'd like to say thanks for your work on this!

KDE uses a generic "Desktop" icon in the launcher rather than the GitHubDesktop icon. This is some combination of #22 and KDE confusing the two icons. I was not able to change it via the Edit Applications... menu until I renamed the icons github-desktop.png (see below for more detail). The correct icon is used elsewhere, so this is not a duplicate of #16

Version

  • GitHub Desktop: 1.1.1 Linux RC
  • Operating system:
    Kubuntu 17.10
  • KDE Plasma 5.10.5

Steps to Reproduce

  1. Install GitHubDesktop via most recent .deb
  2. Open launcher
  3. Find GitHubDesktop entry (generic "Desktop" icon displayed)
  4. Right click launcher > Edit Applications... > Development > GitHubDesktop
  5. Click icon to change > choose desktop icon that matches GitHubDesktop
  6. Exit > Save
  7. Repeat steps 2-3

Expected Behavior

Find GitHubDesktop in launcher, see GitHubDesktop icon.

correct_icon

Actual Behavior

GitHubDesktop icon is generic "desktop" icon, can't be changed via KDE Menu Editor.

generic_icon

Workaround

  1. Rename icons desktop.png to github-desktop.png
sudo mv /usr/share/icons/hicolor/1024x1024/apps/desktop.png /usr/share/icons/hicolor/1024x1024/apps/github-desktop.png
sudo mv /usr/share/icons/hicolor/512x512/apps/desktop.png /usr/share/icons/hicolor/512x512/apps/github-desktop.png
sudo mv /usr/share/icons/hicolor/256x256/apps/desktop.png /usr/share/icons/hicolor/256x256/apps/github-desktop.png
  1. Right click launcher > Edit Applications... > Development > GitHubDesktop
  2. Click icon to change > choose github-desktop icon that matches GitHubDesktop
  3. Exit > Save
@gusbemacbe
Copy link

Not just KDE, it also happens in Gnome and XFCE.

Let me suggest a bit you...

Instead of a PNG format, prefer the SVG format. Instead of Icon=desktop or Icon=github-desktop.png, you should use Icon=github-desktop (without format), so the icon pack themes, like Moka, Numix, Paper, Suru++, etc. can recognise that icon and display it.

Beside, we prefer it open only ONE SAME desktop file, because for example, see the screenshot, when I open the black icon, it opens the purple icon and closes the black icon:

image

@dw5
Copy link

dw5 commented May 25, 2018

BTW Same for Cinnamon (Linux Mint 18.2) It's more packaging related problem I would say...

@oktonion
Copy link

Same for Ubuntu 14.04 LTS with Unity shell. Just default icon.

@shiftkey
Copy link
Owner

shiftkey commented Nov 2, 2018

Rename icons desktop.png to github-desktop.png

I'm aiming to get #80 in for the 1.5.0 release, which renames the package from desktop to github-desktop, and will update these files to use a name:

@shiftkey
Copy link
Owner

I've just published 1.5.0-linux1 that should address the package name issue - please try it out! https://github.com/shiftkey/desktop/releases/tag/release-1.5.0-linux1

@Raymo111
Copy link

@shiftkey this issue seems to be back on 2.9.0?
Screenshot_20210827_004819

@shiftkey
Copy link
Owner

shiftkey commented Sep 7, 2021

@Raymo111 we're still investigating what's changed - please follow long with #559 for updates

@Raymo111
Copy link

Raymo111 commented Sep 7, 2021

Thanks @shiftkey 🙂

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 a pull request may close this issue.

6 participants