ransome1 / sleek

todo.txt manager for Linux, Windows and MacOS, free and open-source (FOSS)
https://github.com/ransome1/sleek/wiki
MIT License
1.29k stars 99 forks source link

[AUR] Tray icon fails to load #580

Open jkdel opened 8 months ago

jkdel commented 8 months ago

Bug Report

App Version: 2.0.2

Platform: Linux

Installation Method: AUR

Bug Description: Fallback icon in systray. Clicking the icon does not open Sleek when the app is reduced to systray.

Steps to Reproduce:

  1. Open Sleek → fallback icon in systray
  2. Activate setting "minimize to taskbar"
  3. Close the window
  4. Clicking the systray icon does nothing

Expected Behavior:

Additional Information: Console log

main.ts: Error: Failed to load image from path '/usr/lib/sleek/app.asar/assets/icons/tray/tray.png'
    at de (/usr/lib/sleek/app.asar/dist/main/main.js:8:260030)
    at /usr/lib/sleek/app.asar/dist/main/main.js:8:270632

The correct path on my system would be '/usr/lib/sleek/assets/icons/tray/tray.png'

ransome1 commented 7 months ago

The correct path on my system would be '/usr/lib/sleek/assets/icons/tray/tray.png'

After installing sleek through AUR, that particular png is located in exactly this location? It would really surprise me, because I think this means sleek had not been packaged during the installation process. Can you check with your terminal or file browser if you can reach that file like this?

Can you check real quick if this also happens if you use the AppImage release? I need to make sure, this only happens with AUR only.

Also, how did you install sleek? With yay?

jkdel commented 7 months ago

Hi,

thanks for the nice app and looking into the issue. I should have provided these details in the first place I guess (hadn't tested the AppImage though):

The AUR packages are the currently available versions there. I've just noticed that the AppImage is more up to date.

The issue occurs on Arch Linux with Gnome and the AppIndicator and KStatusNotifierItem Support shell extension. However, since it works with the AppImage and the path error messages are explicit, I assume this is likely not the issue.

ransome1 commented 7 months ago

I think it has something to do with the way the releases in AUR are being built. Thus this bug report is fully valid.

warmwaffles commented 7 months ago

I can try a rebuild of the package

warmwaffles commented 7 months ago

I updated the AUR to v2.0.2 from v2.0.1

Indeed, the tray icon is not present.

Error creating tray: Error: Failed to load image from path '/usr/lib/sleek.asar/assets/icons/tray/tray.png'
    at de (/usr/lib/sleek.asar/dist/main/main.js:8:260030)
    at e.exports.<anonymous> (/usr/lib/sleek.asar/dist/main/main.js:8:263592)
    at EventEmitter.n (/usr/lib/sleek.asar/dist/main/main.js:2:16042)
    at EventEmitter.emit (node:events:525:35)
    at set store [as store] (/usr/lib/sleek.asar/dist/main/main.js:2:15333)
    at e.exports.set (/usr/lib/sleek.asar/dist/main/main.js:2:14061)
    at IpcMainImpl.Oe (/usr/lib/sleek.asar/dist/main/main.js:8:265091)
    at IpcMainImpl.emit (node:events:513:28)
    at WebContents.<anonymous> (node:electron/js2c/browser_init:2:88759)
    at WebContents.emit (node:events:513:28)
ransome1 commented 7 months ago

I havn't yet found the time to look into it. Somehow we need to include the missing icons into the .asar file during the build process. I won't be able to fix this before 2.0.4 I'm afraid. But it's in the backlog.