deadbeef icon indicating copy to clipboard operation
deadbeef copied to clipboard

OSD Notify does not respect Album art size (px) setting.

Open kierun opened this issue 3 years ago • 7 comments

Steps to reproduce the problem

  1. Enable OSD Notify in plugins.
  2. Set Show Album Art.
  3. Set a maximum size to 256 px -- or whatever.
  4. Load a file with a large cover.
  5. Play the file.

What's going on? Describe the problem in as much detail as possible.

The OSD Notification will appear with a giant image, regardless of the maximum size setting.

Information about the software:

  • Deadbeef version: 1.9.1
  • OS: Fedora 35
  • Package: deadbeef-1.9.1-1.fc35.x86_64

kierun avatar May 30 '22 13:05 kierun

@kierun Hi, which desktop environment are you using?

Oleksiy-Yakovenko avatar May 30 '22 15:05 Oleksiy-Yakovenko

@kierun Hi, which desktop environment are you using?

Awesome WM.

However, it used to work fine till I updated deadbeef the last time.

kierun avatar May 31 '22 07:05 kierun

Does Awesome WM have its own notification daemon? Can you find out which one? I can imagine, awesome does not have its own.. so you have to install something, to get notifications to display at all?

Oleksiy-Yakovenko avatar May 31 '22 15:05 Oleksiy-Yakovenko

It does: naughty. Reading the documentation, I found this thus set it as beautiful.notification_icon_size = 256 and upon re-starting my VM, the icons are now 256x256. I guess my problem is fixed? ¯\_(ツ)_/¯

However, I would assume that the OSD Notification plug-in does the resize of the image before sending it to whatever notification daemon there is. Or is it more of a request to the notification daemon to set the size?

kierun avatar May 31 '22 16:05 kierun

I just checked the code, and the option which is supposed to resize the image before sending it to notification is not functional. Most likely it remained unnoticed when updating to the new artwork plugin. I will have to re-implement it somehow.

Oleksiy-Yakovenko avatar May 31 '22 16:05 Oleksiy-Yakovenko

Apologies for being the bearer of bad news… Do let me know if want me to test things.

Which is the new artwork plugin? I have the Loads album artwork from embedded tags, local directories, or internet services at 2.0.

kierun avatar May 31 '22 16:05 kierun

The new one is the one that comes with deadbeef 1.9.0 and newer.

Oleksiy-Yakovenko avatar May 31 '22 16:05 Oleksiy-Yakovenko

I added the code to scale the images before displaying. It's not perfect, and could be improved slightly to improve caching -- but it should be good enough to start testing it.

Oleksiy-Yakovenko avatar Dec 29 '23 15:12 Oleksiy-Yakovenko