awesomeWM
Notification Icon isn't replaced
@Elv13, sorry I keep hitting you with naughty bugs :smile:
Output of awesome --version:
awesome v4.3-350-ged091838 (Too long)
• Compiled against Lua 5.3.5 (running with Lua 5.3)
• D-Bus support: ✔
• xcb-errors support: ✔
• execinfo support: ✔
• xcb-randr version: 1.6
• LGI version: 0.9.2
How to reproduce the issue:
Trigger a new notification that uses icons (used spotify since it shows album art on each new song) during when the first notifcation is displayed. The text is replaced but icon is not replaced.
NOTE: When first notification is destroyed, the icon is properly set. This does not happen when the first notification is still displayed.
Actual result:
Album Art (Icon) from first notification is still there while text is updated.
Expected result:
Album art (icon) for the recent notification should be the icon and properly replace just like the text.
i still can reproduce it in a bit more limited test case:
if content of the image file changes
then setting notification.icon to the same filename doesn't update the icon
if content of the image file changes then setting notification.icon to the same filename doesn't update the icon
As far as I know, imageboxes in general have this issue.
Should https://github.com/awesomeWM/awesome/blob/87928befe2dc981bfdbca59c0474c0ff5d39544f/lib/naughty/widget/icon.lua#L93 and https://github.com/awesomeWM/awesome/blob/87928befe2dc981bfdbca59c0474c0ff5d39544f/lib/naughty/widget/icon.lua#L160 use load_uncached_silently?
The initial problem reported here is resolved. @actionless, I think the newer issue discussed is not a regression compared to v4.3. So maybe we should close this issue to free up bandwidth in the v4.4 milestone, and refer to PR https://github.com/awesomeWM/awesome/pull/3883 to continue the discussion.
is resolved.
but not yet merged:
https://github.com/awesomeWM/awesome/pull/3883