Fix High CPU utilization #25
Conversation
Replace busy polling on the USR1 signal with a channel [WIP] Subscribe to events from sway to avoid polling
|
Tested with my two presets that use dock, it seems to behave perfectly well. |
|
...apart from the workspaces button not being refreshed. |
|
With the latest commit the workspace button should be refreshed. I also had to fix some functions that handled showing images, otherwise in my debug environment only app images would be displayed ... Btw it seems like the sway ipc dependency doesn't map the subscribe event correctly for workspaces, so my solution needs to re-query the workspace information from sway to get the number of the focused workspace. |
|
Workspaces button behaves well for me. There's still one minor issue: unpinned items do not disappear from the dock right after being unpinned. You need to start the application and then close it. Especially well visible if you keep the dock permanently on the screen. |
|
Pin/Unpin actions should now work |
|
Yes, everything seems to work just well. Thank you for your work. |
This PR fixes Issue #24. Now tasks are only loaded if changes are detected (opening / closing windows or modifying the UI). I have not fully tested everything, so it is possible that not all state changes are tracked correctly and may result in an inconsistent state of the user interface.