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

Recent builds have missing icons and weird window dimensions #1392

Closed
Samueru-sama opened this issue Nov 3, 2024 · 9 comments
Closed

Recent builds have missing icons and weird window dimensions #1392

Samueru-sama opened this issue Nov 3, 2024 · 9 comments

Comments

@Samueru-sama
Copy link

Samueru-sama commented Nov 3, 2024

Issue description

Here is how the notifications used to look on my system using the last stable build of dunst (upper right corner):

image

Here is how the look using dunst-git (and also happens with this appimage build from the master branch)

image

Icons are gone and the notifications have weird window dimensions.

I noticed dunst gave me this warning: MESSAGE: Using legacy offset syntax NxN, you should switch to the new syntax (N, N) so what my config is now obsolete? so I tested removing it and launching dunst with defaults.

image

Also when my music player sends notifications I get this error:

image

So it seems the issue is just the icons and the window size is that I need to update my config, because with default settings this is how the stable build looks (has working icons):

image

Installation info

  • Version: v1.11.0-92-g9d3c91a
  • Install type: Dunst-git aur package (and also happens with an appimage that I make when built on the master branch)
  • Window manager / Desktop environment: i3-wm
  • Distro: Artix

Dunstrc here

@bynect
Copy link
Member

bynect commented Nov 3, 2024

Hello.

The icons and size are actually correct. This is yet to be properly documented (it is testing). But if you have any suggestions on how to make it painless to switch and document it would be great.

Rationale for not including icons in the backed in config: some systems don't have the icons requested and you have to do extra steps to remove the icon. This way if you want them you can easily add to dunstrc, otherwise they will be without icon

The height issue is due to dynamic height. Now the height can be variable between (min, max) just like the width. If you provide a single height it will be read as a fixed height (min=max). This means that you can fix your problem by doing
height = (0 , oldheight)

The last thing about missing icons messages. I don't know if it is normal. Are the deadbeef icons present in the themes?
If the icons is not found by dunst this logging seems right though

@Samueru-sama
Copy link
Author

Samueru-sama commented Nov 3, 2024

Hello.

The icons and size are actually correct. This is yet to be properly documented (it is testing). But if you have any suggestions on how to make it painless to switch and document it would be great.

The height issue is due to dynamic height. Now the height can be variable between (min, max) just like the width. If you provide a single height it will be read as a fixed height (min=max). This means that you can fix your problem by doing height = (0 , oldheight)

Thanks, I was able to fix the height issue.

The last thing about missing icons messages. I don't know if it is normal. Are the deadbeef icons present in the themes? If the icons is not found by dunst this logging seems right though

The icon of deadbeef is not in the Adwaita icon theme, however I have enable_recursive_icon_lookup enabled which means it should still try to find it right?

With that said, I don't remember ever seeing the deadbeef icon on my notification and it still shows the album art of the current playing track, so maybe that's normal? however the error about the missing icon does not happen with the stable release.

And also the default icons I used to get, which is that light bulb and a warning sign for critical notifications are gone.

EDIT:

Rationale for not including icons in the backed in config: some systems don't have the icons requested and you have to do extra steps to remove the icon. This way if you want them you can easily add to dunstrc, otherwise they will be without icon

They are still missing with my existing config.

@bynect
Copy link
Member

bynect commented Nov 4, 2024

you should be able to add to

[urgency_normal]
default_icon = dialog-information
...

@bynect
Copy link
Member

bynect commented Nov 4, 2024

I will add this to the default dunstrc to keep backward compatibility of the config

@bynect
Copy link
Member

bynect commented Nov 4, 2024

The icon of deadbeef is not in the Adwaita icon theme, however I have enable_recursive_icon_lookup enabled which means it should still try to find it right?

With that said, I don't remember ever seeing the deadbeef icon on my notification and it still shows the album art of the current playing track, so maybe that's normal? however the error about the missing icon does not happen with the stable release.

if it is in any of the themes you have installed it should find it. however that error means that it doesn't find anywhere. so it is provided by an actual theme you have installed? The changes to icon lookup where made in #1365 btw

@bynect
Copy link
Member

bynect commented Nov 4, 2024

for the problem of icon error that you found. it is true and it is actually an error. the only change I made is that I before it was an INFO message (so it was hidden). Now it is a warning.
but I think there is something wrong with the theme search (okay I didn't put the right theme in the conf) Further work is required. Thanks for reporting 👍🏻
In the meantime you can check the dunstrc in #1393

@Samueru-sama
Copy link
Author

you should be able to add to

[urgency_normal]
default_icon = dialog-information
...

Oh now I get the icons back, but that's odd, with the old dunst I still had #default_icon = /path/to/icon and somehow it still showed those icons? Anyways now it works which is the important part!

if it is in any of the themes you have installed it should find it. however that error means that it doesn't find anywhere. so it is provided by an actual theme you have installed? The changes to icon lookup where made in #1365 btw

I'm very sorry, it seems I don't have a deadbeef icon in my themes, I'm actually surprised because I have the Papirus icon theme and that one usually has everything.

I don't remember ever seeing the DeaDBeeF icon in my notifications, it instead sends the album art for display, so the only difference between before and now is that now the warning works and not that the icon can't be find 😅

for the problem of icon error that you found. it is true and it is actually an error. the only change I made is that I before it was an INFO message (so it was hidden). Now it is a warning.

Yeah it was this.

I think this issue can be closed now right?

@bynect
Copy link
Member

bynect commented Nov 4, 2024

Oh now I get the icons back, but that's odd, with the old dunst I still had #default_icon = /path/to/icon and somehow it still showed those icons? Anyways now it works which is the important part!

because before the icon names were hardcoded in dunst source code. now they can easily be specified in the dunstrc if needed.

I think this issue can be closed now right?

👍🏻

@bynect
Copy link
Member

bynect commented Dec 7, 2024

For future reference, read the RELEASE_NOTES for 1.12 on how to fix the height change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants