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 696380 - curtain falls slower
curtain falls slower
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: lock-screen
3.7.x
Other Linux
: Normal trivial
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-03-22 12:08 UTC by Jakub Steiner
Modified: 2013-03-26 08:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenShield: Go back to a shorter animation when locking manually (1016 bytes, patch)
2013-03-25 17:26 UTC, Rui Matos
committed Details | Review

Description Jakub Steiner 2013-03-22 12:08:45 UTC
I'm guessing the increase of the transition time was a workaround to the choppiness introduced with the desaturate and blur filters, but the curtain fell much faster in 3.6 than 3.8. We should keep the transition time of 3.6.
Comment 1 Rui Matos 2013-03-22 13:10:02 UTC
Apparently there was a designer OK for the change from 0.3 to 0.8 some weeks ago, see https://bugzilla.gnome.org/show_bug.cgi?id=691964#c35 and following comments. 0.3 always felt better to me...

Also, making it faster would help to paper over the fact that we're now abruptly hiding the windows when the shield animation starts. This was caused by https://git.gnome.org/browse/gnome-shell/commit/?id=b328fd7aed4455b6b7378480baa2d604806f0bea and doesn't look like there's an easy and quick fix.
Comment 2 Matthias Clasen 2013-03-24 17:14:33 UTC
now that we have the blur and desaturation removed, should we consider this ?
Comment 3 Rui Matos 2013-03-25 17:26:09 UTC
Created attachment 239800 [details] [review]
screenShield: Go back to a shorter animation when locking manually

Increasing it was mostly to paper over jerkiness which shouldn't be a
problem anymore since we removed the desaturate and blur effects.
Comment 4 Florian Müllner 2013-03-25 19:19:46 UTC
Review of attachment 239800 [details] [review]:

Sure, if designers + r-t agree ...
Comment 5 Matthias Clasen 2013-03-25 19:31:35 UTC
I'll give my +1 here, go get a second one on the list...
Comment 6 Rui Matos 2013-03-26 08:38:45 UTC
Got release team approval:
https://mail.gnome.org/archives/release-team/2013-March/msg00286.html

Attachment 239800 [details] pushed as 9d18a2d - screenShield: Go back to a shorter animation when locking manually