zidoro / pomatez

Stay Focused. Take a Break.
https://zidoro.github.io/pomatez/
MIT License
4.06k stars 171 forks source link

Bug Report: there's no shortcut icon on Fedora 32 with Gnome #106

Open hunkjazz opened 4 years ago

hunkjazz commented 4 years ago

Describe the bug When I press windows key to see installed softwares, this app is there but with gnome default icon.

Screenshots icon

Expected behavior I assuming the app should have an icon.

Desktop (please complete the following information):

roldanjr commented 4 years ago

I didn't see that happening to my Manjaro. It's rare, but I advice if possible just install the app using snap.

If the issue still persist even using the snap package please inform me. Thank you @hunkjazz .

hunkjazz commented 4 years ago

It persists only for the .rpm release.

Snap package works fine.

vladspirin commented 4 years ago

I have the same problem: Screenshot. OS: Fedora 32 Workstation

roldanjr commented 4 years ago

@vladspirin I honestly don't know why it's happening.

That is why I personally suggest using the snap package instead of others when using on Linux machines.

Using snap you will also always up-to-date because .rpm doesn't support auto-updates for Linux as far as I know.

If possible just install the app using SnapStore.

Maybe @sekwah41 have an idea. I don't know.

Anyway, thanks for understanding.

sekwah41 commented 4 years ago

I'll add a task this weekend to setup a fedora VM and give it a test. I assume it'll be something simple e.g. either the file on the desktop has a missing tag for the image or is pointing to the wrong location for some reason.

roldanjr commented 4 years ago

Thank you so much @sekwah41.

You will find the configuration here at app/main/package.json.

I initially tried to fix it up, but I can't confirm if it solves the problem. Thank you

roldanjr commented 4 years ago

@sekwah41 If you have still extra time, I would like to ask a favor if possible please include in your test if issue #127 has been fixed. Thank you friend.

vladspirin commented 4 years ago

@roldanjr Thanks for your answer. Are you planning to release a "flatpak" version?

roldanjr commented 4 years ago

Maybe soon @vladspirin. I used Electron Builder to distribute the app and it doesn't support flatpak yet. But it's a great idea having that.

sekwah41 commented 4 years ago

image can confirm on fedora 32 using the rpm

sekwah41 commented 4 years ago

image taking a look a the documentation, ill make a pr fixing both the Linux issues

Edit: One issue, you may have fixed the other