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 661077 - messageTray: use BoxPointer for summary item titles
messageTray: use BoxPointer for summary item titles
Status: RESOLVED OBSOLETE
Product: gnome-shell
Classification: Core
Component: message-tray
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 670769 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-06 12:55 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2012-11-02 15:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
messageTray: use BoxPointer for summary item titles (16.15 KB, patch)
2011-10-06 12:55 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
messageTray: use BoxPointer for summary item titles (16.87 KB, patch)
2011-10-07 20:41 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
messageTray: use BoxPointer for summary item titles (16.98 KB, patch)
2011-10-24 19:02 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
screenshot of patch effects (12.97 KB, image/png)
2012-02-27 17:19 UTC, Steven Brown
  Details

Description Jasper St. Pierre (not reading bugmail) 2011-10-06 12:55:00 UTC
Some people have requested it, so I rebased my patch from bug #636298
and re-filed it.
Comment 1 Jasper St. Pierre (not reading bugmail) 2011-10-06 12:55:02 UTC
Created attachment 198429 [details] [review]
messageTray: use BoxPointer for summary item titles

The accordion resizing widget was a bit finnicky for some, so this may be a bit
better. This also cleans up a lot of the implementation because of the constant
moving of actors meant that the summaries needed to keep track of a lot of random
state.
Comment 2 Matthias Clasen 2011-10-06 13:29:34 UTC
using the right component might help to get feedback from the right people...
Comment 3 Matthias Clasen 2011-10-07 12:25:30 UTC
Also, the bug title has no indication at all that this is about a significant user experience change...
Comment 4 Jasper St. Pierre (not reading bugmail) 2011-10-07 14:49:24 UTC
(In reply to comment #3)
> Also, the bug title has no indication at all that this is about a significant
> user experience change...

Feel free to rename it.
Comment 5 Jasper St. Pierre (not reading bugmail) 2011-10-07 20:41:51 UTC
Created attachment 198568 [details] [review]
messageTray: use BoxPointer for summary item titles

The accordion resizing widget was a bit finnicky for some, so this may be a bit
better. This also cleans up a lot of the implementation because of the constant
moving of actors meant that the summaries needed to keep track of a lot of random
state.



Fix notification icons getting "squashed", and the source title boxpointer
appearing under other things, as well as general cleanup and related bugs.

Just a note: this is still only a proof of concept. If we were to decide to go
this route (I still think it's the "correct" route, TBH), then I need to go
through all the comments and code and try to take out anything incorrect or
extraneous.
Comment 6 Jasper St. Pierre (not reading bugmail) 2011-10-24 19:02:58 UTC
Created attachment 199858 [details] [review]
messageTray: use BoxPointer for summary item titles

The accordion resizing widget was a bit finnicky for some, so this may be a bit
better. This also cleans up a lot of the implementation because of the constant
moving of actors meant that the summaries needed to keep track of a lot of random
state.



Fix an issue where the box pointer wouldn't show after clicking on the summary item while it was open.
Comment 7 xcorex 2011-11-28 15:44:40 UTC
Jasper St. Pierre,
Thank you a lot! You don't know how your patch made me happy!
Now I have no complain about the messaging tray.
Comment 8 Steven Brown 2012-02-24 20:30:27 UTC
I *much* prefer this behaviour. Thank you for the patch!
Comment 9 Florian Müllner 2012-02-27 10:53:36 UTC
*** Bug 670769 has been marked as a duplicate of this bug. ***
Comment 10 Steven Brown 2012-02-27 17:19:20 UTC
Created attachment 208505 [details]
screenshot of patch effects
Comment 11 Steven Brown 2012-03-11 17:55:12 UTC
There is an extension which looks to do the same thing.
https://extensions.gnome.org/extension/210/no-messagetray-text/

The discussion there may be relevant.
Comment 12 Florian Müllner 2012-11-02 15:06:48 UTC
Obsoleted by the message tray updates in 3.6.