GNOME Bugzilla – Bug 654845
Move tooltips under pop-up bubbles
Last modified: 2011-12-22 10:33:52 UTC
Created attachment 192180 [details] bug demonstration Please see attached screenshot. When application name is long and I click right mouse button on the application icon in overview mode, the tooltip saying application name is above the pop-up bubble. That means I can't read some of its text. The tooltip should probably go away or be placed below the bubble, so that I can read all the text. There is no easy workaround currently to see the whole text for such applications. gnome-shell-3.0.2-4.fc15.x86_64 Fedora 15
Can confirm this here. gnome-shell 3.2.0 Ubuntu 11.10
Created attachment 199182 [details] Demonstrates tooltip overlaying text in bubbles
Still valid in Gnome 3.2.
A better and simpler solution would be to not show the tooltips when the menu is displayed.
Created attachment 202747 [details] [review] [Dash] Don't show the icon's tooltips when the menu is displayed (#654845).
The attached patch fixes the bug using Allan's hint.
*** Bug 649221 has been marked as a duplicate of this bug. ***
(In reply to comment #4) > A better and simpler solution would be to not show the tooltips when the menu > is displayed. Note that the current behavior of showing tooltips while the menu is up was done on explicit designer request (https://bugzilla.gnome.org/show_bug.cgi?id=642871#c3). So before moving forward here, I'd like to get some input from Jon.
Created attachment 202863 [details] [review] [Dash] Don't show the icon's tooltips when the menu is displayed (#654845).
With the previous patch there was a problem: the hover highlight disappears once the popup menu shows up. I've attached a new patch that fixes this. Waiting for Jon's input.
(In reply to comment #10) > With the previous patch there was a problem: the hover highlight disappears > once the popup menu shows up. > I've attached a new patch that fixes this. > Waiting for Jon's input. I have discussed this with Jon and the other designer types. I think there's general agreement that we should hide the tooltips when the menu is open. I've also tried your patch, which definitely fixes the bug. I wonder if we can improve the behaviour a little bit though. The patch seems to hide the tooltip when you click the application icon. I actually quite like this when you are single clicking the app to launch or switch to it. Unfortunately it does mean that the tooltip disappears long before the menu turns up on a long click, however. A few design recommendations, therefore: * On long click, leave the tooltip open after the click begins until just before the menu opens. * On short/single click remove the tooltip straight away. * When remove the tooltip use a very quick fade out. Hope that makes sense!
Allan, i think it makes sense...i'm on it! :)
Created attachment 203411 [details] [review] New Patch This new patch solves the problem that Allan said and improves the tooltip behaviour as proposed. P.S. I've also added a fade-in animation for the tooltip...tell me if you like it.
(In reply to comment #13) > Created an attachment (id=203411) [details] [review] > New Patch > > This new patch solves the problem that Allan said and improves the tooltip > behaviour as proposed. > > P.S. I've also added a fade-in animation for the tooltip...tell me if you like > it. Thanks Stefano, this is great! Definitely an improvement. I like the fade in. The only thing I would suggest is making the fade a bit quicker. 150 worked for me. Oh, and you seem to have attached a diff as opposed to a patch.
Created attachment 203435 [details] [review] Updated Patch Here it is the patch (sorry for the previous diff).
(In reply to comment #15) > Created an attachment (id=203435) [details] [review] > Updated Patch > > Here it is the patch (sorry for the previous diff). It seems that the patch in bug 666166 touches on the same ground as this one. I've asked its author to ensure that he incorporates the work that has been done here.
I can confirm i integrated Cando's concept into bug 666166. Awesome work as usual Cando :)
The patch for bug 666166 has fixed this. Thanks for your work, Cando!