After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 544592 - Mouse-over notification bubble should only hide after a timeout
Mouse-over notification bubble should only hide after a timeout
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: User Interface
git master
Other All
: Normal minor
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
: 536534 592133 (view as bug list)
Depends on:
Blocks: 544590
 
 
Reported: 2008-07-24 19:32 UTC by Michał Sawicz
Modified: 2009-08-18 23:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Delay Hiding of Notification Bubble (772 bytes, patch)
2009-04-07 05:38 UTC, Neil Loknath
none Details | Review
needs to prevent addition of delay if one exists (1.38 KB, patch)
2009-04-09 02:15 UTC, Neil Loknath
none Details | Review
Don't hide the track info popup while it has focus (2.48 KB, patch)
2009-05-31 08:16 UTC, Alexander Kojevnikov
committed Details | Review

Description Michał Sawicz 2008-07-24 19:32:39 UTC
The mouse-over notification hides as soon as mouse moves away from the tray icon.
It should hide after a timeout, more like the on-track-change notification bubble does.
It should also allow to seek, show bigger cover (as is the case with the small cover art in the main UI) and show / allow changing of the song's rating (more here: #544590)

Other information:
Comment 1 Michael Martin-Smucker 2008-08-25 02:22:47 UTC
It's true, the notification has a seek bar which makes you think you would be able to use it for something, but mousing over the notification makes the notification go away.  Seems strange.
Comment 2 Michael Martin-Smucker 2009-02-18 23:38:05 UTC
Actually, it looks like the most significant part of this bug (notification should stay up longer so you can play with the seek bar) is already covered in bug #536534 .  

Does that make this one a dupe?
Comment 3 Alexander Kojevnikov 2009-02-20 07:42:52 UTC
(In reply to comment #2)
> Does that make this one a dupe?

I think so, looks like this one can be closed.

Comment 4 Gabriel Burt 2009-02-20 18:44:27 UTC
They are actually about different ballons - the one you get when a song changes and the one you get when you mouseover the icon.  Let's keep them both open for now.
Comment 5 Neil Loknath 2009-04-07 05:38:31 UTC
Created attachment 132237 [details] [review]
Delay Hiding of Notification Bubble

this is a patch for /svn/banshee/src/Extensions/Banshee.NotificationArea/Banshee.NotificationArea/X11NotificationAreaBox.cs. The delay I used is up for debate: 10 seconds? This patch is tied to http://bugzilla.gnome.org/show_bug.cgi?id=544590. However, even without the patch for #544590, this one is still useful for song seeking. #544590 still needs work to apply rating changes to the database.
Comment 6 Neil Loknath 2009-04-09 02:15:43 UTC
Created attachment 132381 [details] [review]
needs to prevent addition of delay if one exists

patch was incomplete. this update is also included in patch for bug http://bugzilla.gnome.org/show_bug.cgi?id=544590
Comment 7 Alexander Kojevnikov 2009-05-31 08:16:19 UTC
Created attachment 135648 [details] [review]
Don't hide the track info popup while it has focus

(Neil wrote in bug 544590)
> - instead of delaying hiding of tooltip for 10 seconds, delay it long enough
> (maybe 1 sec) so that the user can set the mouse pointer over the tooltip.
> - when tooltip has focus, keep it up for the duration that it is in focus (have
> to make sure it updates if track changes!!).
> - as soon as the tooltip no longer has focus, hide it.

Neil, the patch should do this.

Cheers,
Alex
Comment 8 Alexander Kojevnikov 2009-05-31 08:35:03 UTC
(In reply to comment #4)
> They are actually about different ballons - the one you get when a song changes
> and the one you get when you mouseover the icon.  Let's keep them both open for
> now.
> 

Actually they are about the same balloon - TrackInfoPopup. Here's the commit Alexander mentions in bug 536534: http://git.gnome.org/cgit/banshee/commit/?id=8ad8e0f9d10af8da7207f983a4f0a7ebc633ce62

The other balloon is shown when the currently playing track changes and it has no seek slider at all, only the track information and the skip button (in Ubuntu 9.04 it doesn't even have the button because of the new NotifyOSD)
Comment 9 Alexander Kojevnikov 2009-05-31 08:37:31 UTC
*** Bug 536534 has been marked as a duplicate of this bug. ***
Comment 10 Neil Loknath 2009-06-06 05:23:00 UTC
Nice, Alex. It's working well for me. I'll try and use this for the ratings patch I made for http://bugzilla.gnome.org/show_bug.cgi?id=544590.
Comment 11 Neil Loknath 2009-06-06 06:03:08 UTC
Small nitpick:
If a drop down menu is left open and the mouse focus is moved over to the tray icon to bring up the pop up, moving the focus to the pop up does not keep it open.

By the way, bug 544590 has been updated with a new patch.

Enjoy,
Neil
Comment 12 Alexander Kojevnikov 2009-06-06 14:39:54 UTC
(In reply to comment #11)
> Small nitpick:
> If a drop down menu is left open and the mouse focus is moved over to the tray
> icon to bring up the pop up, moving the focus to the pop up does not keep it
> open.

Hmm, apparently the pop up never receives the EnterNotifyEvent after the drop down menu is opened. I guess this is intentional, try to open a drop down menu in any gtk app and move the mouse within the application window - the controls don't react to the mouse movements at all.

The same is true for the pop ups from the notification area. For example, if I open Totem, right-click in its window, then move the mouse over the Banshee's notification icon - the pop up does not appear.

The only exception is Banshee itself, if the drop down is open inside Banshee, either in the main window or in the notification area icon, then the pop up is shown.

I will open a new bug for this.
Comment 13 Alexander Kojevnikov 2009-06-06 14:45:31 UTC
(In reply to comment #12)
> I will open a new bug for this.

Filed as bug 584997.
Comment 14 Gabriel Burt 2009-06-09 17:11:09 UTC
Hey Alexander,

Why do you need to do the Intersects check - why can't you always assume the LeaveNotify should start the timeout?  Did you find in your testing that there were cases it didn't intersect?
Comment 15 Alexander Kojevnikov 2009-06-10 00:14:39 UTC
(In reply to comment #14)
> Hey Alexander,
> 
> Why do you need to do the Intersects check - why can't you always assume the
> LeaveNotify should start the timeout?  Did you find in your testing that there
> were cases it didn't intersect?
> 

Yep, when the mouse is over the ClassicTrackInfo display, the popup receives a LeaveNotifyEvent.
Comment 16 Gabriel Burt 2009-06-16 00:33:21 UTC
Ok, thanks
Comment 17 Alexander Kojevnikov 2009-06-16 00:53:58 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 18 Bertrand Lorentz 2009-08-17 22:43:48 UTC
*** Bug 592133 has been marked as a duplicate of this bug. ***
Comment 19 Tor Bechmann Sørensen 2009-08-17 23:28:11 UTC
Hey Bertrand thanks for attending to my bug so quickly.

My bug was about internet radio, and that the slider doesn't make sense in that context (as you can't jump back and forth in the stream). I am wondering, does  Alexander's fix account for this special case?
Comment 20 Alexander Kojevnikov 2009-08-18 23:39:19 UTC
(In reply to comment #19)
> Hey Bertrand thanks for attending to my bug so quickly.
> 
> My bug was about internet radio, and that the slider doesn't make sense in that
> context (as you can't jump back and forth in the stream). I am wondering, does 
> Alexander's fix account for this special case?

Nope, it doesn't. But as Bertrand mentioned in bug 592133:

> While streaming, the slider in the popup behaves just like the slider
> in the main UI.