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 677210 - Don't show the message tray background when a banner pops up
Don't show the message tray background when a banner pops up
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: message-tray
3.3.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-05-31 17:17 UTC by Allan Day
Modified: 2012-07-08 17:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch implementing the feature (5.12 KB, patch)
2012-07-08 12:20 UTC, Ana Risteska
committed Details | Review

Description Allan Day 2012-05-31 17:17:56 UTC
Currently, the message tray background (the gradient at the bottom of the screen) is displayed when a banner pops up. It would be better not to show the gradient - the background should only be displayed when the tray is properly open.

Some mockups:
 * http://www.youtube.com/watch?v=14z4wdgNF9g
 * http://git.gnome.org/browse/gnome-shell-design/plain/mockups/static/notifications-chat-banner.png
Comment 1 Ana Risteska 2012-07-08 12:20:34 UTC
Created attachment 218266 [details] [review]
Patch implementing the feature

Hi!
Here is a proposed patch for the bug.
Comment 2 Marina Zhurakhinskaya 2012-07-08 17:02:44 UTC
Review of attachment 218266 [details] [review]:

Looks good to me. Committed with small changes to the commit message. Thanks for the patch, Ana!
Comment 3 Jasper St. Pierre (not reading bugmail) 2012-07-08 17:17:00 UTC
I don't understand this. How does this relate to the gradient being there or not?

Seems we should just have a style class that gets added/removed in banner vs. summary mode that adds the gradient.

(Also, there's a style nit with the space after a unary - operator)
Comment 4 Jasper St. Pierre (not reading bugmail) 2012-07-08 17:18:16 UTC
Oh, is the idea that there's no longer an entire message tray too, so clicks will by-pass it? If so, yay!

Does this mean that simply moving my mouse to the bottom (but not over the notification) will no longer expand the notification? Will it at least keep the notification from timing out?
Comment 5 Allan Day 2012-07-08 17:56:09 UTC
(In reply to comment #4)
> Oh, is the idea that there's no longer an entire message tray too, so clicks
> will by-pass it? If so, yay!

The gradient won't be displayed when a banner is up, so no - it won't intercept clicks in this case.

> Does this mean that simply moving my mouse to the bottom (but not over the
> notification) will no longer expand the notification? Will it at least keep the
> notification from timing out?

For now, placing the pointer on the bottom screen edge over a banner should expand the banner and stop it from disappearing. The longer term plan for this can be found in bug 677217.
Comment 6 Allan Day 2012-07-08 17:56:29 UTC
This is fantastic, thanks Ana!