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 706841 - screenShield: Don't fade in the lock dialog
screenShield: Don't fade in the lock dialog
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-08-26 21:47 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2013-08-26 22:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenShield: Don't fade in the lock dialog (3.33 KB, patch)
2013-08-26 21:47 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description Jasper St. Pierre (not reading bugmail) 2013-08-26 21:47:32 UTC
See patch. This causes flickering for me on Intel when locking the screen.
Comment 1 Jasper St. Pierre (not reading bugmail) 2013-08-26 21:47:34 UTC
Created attachment 253186 [details] [review]
screenShield: Don't fade in the lock dialog

We slide the shield over it, so the animation is rarely seen, and
since no other actor is under the lock screen, the not-cleared stage
can show through, causing weird issues when trying to blend.
Comment 2 drago01 2013-08-26 21:51:46 UTC
Review of attachment 253186 [details] [review]:

OK, having an animation that is rarely seen also isn't a good idea from a performance POV so kill it.
Comment 3 Jasper St. Pierre (not reading bugmail) 2013-08-26 21:53:13 UTC
Attachment 253186 [details] pushed as b7b1260 - screenShield: Don't fade in the lock dialog
Comment 4 Giovanni Campagna 2013-08-26 21:56:14 UTC
Wait a minute...

This was for gdm startup (it was in the constructor), not for lock, so it was over the cleared stage and was actually very visible.
Comment 5 Jasper St. Pierre (not reading bugmail) 2013-08-26 21:59:02 UTC
I removed both transitions: the one on startup and the one on lock. I thought the gdm login dialog one was handled by the new opacity tween in layout.js. I can add back the one in the constructor if you want.
Comment 6 Giovanni Campagna 2013-08-26 22:02:22 UTC
(In reply to comment #5)
> I removed both transitions: the one on startup and the one on lock. I thought
> the gdm login dialog one was handled by the new opacity tween in layout.js. I
> can add back the one in the constructor if you want.

The "new opacity tween in layout.js"?
Comment 7 Jasper St. Pierre (not reading bugmail) 2013-08-26 22:07:16 UTC
hm, I remember reviewing or writing a patch that faded the screenShieldGroup in the greeter animation. Guess it didn't go in.
Comment 8 Jasper St. Pierre (not reading bugmail) 2013-08-26 22:19:12 UTC
fixed in 80ab28b