From 5ad7db722d1cbf42d803ae9c6989f4d9804369b9 Mon Sep 17 00:00:00 2001 From: Giovanni Campagna Date: Wed, 3 Oct 2012 23:13:25 +0200 Subject: [PATCH] ScreenShield: don't allow cancelling the curtain by pressing esc twice in the dialog If esc is pressed twice in succession in the unlock dialog, the curtain is cancelled, but the dialog is cleared after the first esc cancels it, and it's not destroyed and recreated. https://bugzilla.gnome.org/show_bug.cgi?id=685441 --- js/ui/screenShield.js | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/js/ui/screenShield.js b/js/ui/screenShield.js index 0e5557eb0..4674e6578 100644 --- a/js/ui/screenShield.js +++ b/js/ui/screenShield.js @@ -439,6 +439,17 @@ const ScreenShield = new Lang.Class({ _onLockScreenKeyRelease: function(actor, event) { let symbol = event.get_key_symbol(); + // Do nothing if the lock screen is not fully shown. + // This avoids reusing the previous (and stale) unlock + // dialog if esc is pressed while the curtain is going + // down after cancel. + // Similarly, don't bump if the lock screen is not showing or is + // animating, as the bump overrides the animation and would + // remove any onComplete handler. + + if (this._lockScreenState != MessageTray.State.SHOWN) + return false; + if (symbol == Clutter.KEY_Escape || symbol == Clutter.KEY_Return || symbol == Clutter.KEY_KP_Enter) { @@ -447,11 +458,7 @@ const ScreenShield = new Lang.Class({ return true; } - // Don't bump if the lock screen is not showing or is - // animating, as the bump overrides the animation and would - // remove any onComplete handler - if (this._lockScreenState == MessageTray.State.SHOWN) - this._bumpLockScreen(); + this._bumpLockScreen(); return true; },