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 682271 - Increase the opacity of the message tray lightbox
Increase the opacity of the message tray lightbox
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: message-tray
3.5.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-08-20 15:03 UTC by Allan Day
Modified: 2012-08-20 20:51 UTC
See Also:
GNOME target: ---
GNOME version: 3.5/3.6


Attachments
theme: Put back the transition from overview to desktop for the message tray (779 bytes, patch)
2012-08-20 19:00 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review
messageTray: Put the lightbox opacity down (1.09 KB, patch)
2012-08-20 19:00 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description Allan Day 2012-08-20 15:03:58 UTC
It's a bit dark right now; this makes the message tray mode rather disruptive, and results in the popups not having enough contrast with the background.

It seems that the opacity is currently set to 0.4. 0.2 would be a better value.
Comment 1 Jasper St. Pierre (not reading bugmail) 2012-08-20 19:00:14 UTC
Created attachment 221876 [details] [review]
theme: Put back the transition from overview to desktop for the message tray
Comment 2 Jasper St. Pierre (not reading bugmail) 2012-08-20 19:00:40 UTC
Created attachment 221877 [details] [review]
messageTray: Put the lightbox opacity down

Whoops; attached the wrong patch to this bug. But they're both simple.
Comment 3 Marina Zhurakhinskaya 2012-08-20 20:27:21 UTC
Review of attachment 221876 [details] [review]:

::: data/theme/gnome-shell.css
@@ +1167,3 @@
     background: #2e3436 url(message-tray-background.png);
     background-repeat: repeat;
+    transition-duration: 250ms;

the correct syntax is "250", not "250ms"
Comment 4 Marina Zhurakhinskaya 2012-08-20 20:29:17 UTC
Review of attachment 221877 [details] [review]:

lgtm
Comment 5 Jasper St. Pierre (not reading bugmail) 2012-08-20 20:51:26 UTC
Attachment 221876 [details] pushed as 75ed427 - theme: Put back the transition from overview to desktop for the message tray
Attachment 221877 [details] pushed as d1e35d1 - messageTray: Put the lightbox opacity down