Here's how rhythmbox should go about this case when second song lacks
cover-art (pseudo-code):
// create a new bubble for first song
image_file = get_cover_art_image (song_one);
if (image_file)
n = notification_new ("Song one", "Artist one, Album one", image_file);
else
n = notification_new ("Song one", "Artist one, Album one", " "); //
note the space between the last double-quote pair
notification_show (n);
// update current bubble
image_file = get_cover_art_image (song_two);
if (image_file)
notification_update (n, "Song two", "Artist two, Album two",
image_file);
else
notification_update (n, "Song two", "Artist two, Album two", " "); //
note the space between the last double-quote pair
notification_show (n);
According to you example-case the first song will have a cover-image,
thus the if-branch will be executed (resulting in an icon-summary-body
layout case). For the second song the else-branch will be executed,
because it lacks a cover-image. Then the filename will be a simple
single space (and result in a summary-body layout case). This is needed
for notify-osd to be able to determine the bubble-layout and distinguish
update- from append-cases.
--
Rhythmbox Notification Bubble shows wrong cover art
https://bugs.launchpad.net/bugs/451086
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
