From 8395f8bad5698950c6ef0ef53c0383ddaa2c527b Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Fri, 29 Nov 2019 12:43:18 -0300 Subject: [PATCH] sessionMode: Remove lock-screen mode Now that the screen shield is gone (at least, as it used to be), the corresponding session mode is not necessary anymore as well. Remove the 'lock-screen' session mode, and the corresponding CSS. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/872 --- data/theme/gnome-shell-sass/_common.scss | 10 +++------- js/ui/screenShield.js | 11 +---------- js/ui/sessionMode.js | 13 ------------- 3 files changed, 4 insertions(+), 30 deletions(-) diff --git a/data/theme/gnome-shell-sass/_common.scss b/data/theme/gnome-shell-sass/_common.scss index f765a0d77..02478dc99 100644 --- a/data/theme/gnome-shell-sass/_common.scss +++ b/data/theme/gnome-shell-sass/_common.scss @@ -826,8 +826,7 @@ StScrollBar { font-feature-settings: "tnum"; &.unlock-screen, - &.login-screen, - &.lock-screen { + &.login-screen { background-color: transparent; } @@ -845,7 +844,7 @@ StScrollBar { -panel-corner-border-color: lighten($selected_bg_color,5%); } - &.lock-screen, &.login-screen, &.unlock-screen { + &.login-screen, &.unlock-screen { -panel-corner-radius: 0; -panel-corner-background-color: transparent; -panel-corner-border-color: transparent; @@ -879,8 +878,7 @@ StScrollBar { .system-status-icon { icon-size: 1.09em; padding: 0 5px; } .unlock-screen &, - .login-screen &, - .lock-screen & { + .login-screen & { color: lighten($fg_color, 10%); &:focus, &:hover, &:active { color: lighten($fg_color, 10%); } } @@ -2097,8 +2095,6 @@ $_unlockdialog_shadow: 0px 0px 6px rgba(0, 0, 0, 0.726); .unlock-dialog-notification-count-text { padding: 0px 0px 0px 12px; } -#panel.lock-screen { background-color: transparentize($osd_bg_color, 0.5); } - .screen-shield-background { //just the shadow, really background: black; box-shadow: 0px 2px 4px transparentize(black,0.6); diff --git a/js/ui/screenShield.js b/js/ui/screenShield.js index b7dd32dbd..94e379bca 100644 --- a/js/ui/screenShield.js +++ b/js/ui/screenShield.js @@ -336,9 +336,6 @@ var ScreenShield = class { } _hideLockScreenComplete() { - if (Main.sessionMode.currentMode == 'lock-screen') - Main.sessionMode.popMode('lock-screen'); - this._lockScreenState = MessageTray.State.HIDDEN; this._lockScreenGroup.hide(); @@ -443,9 +440,6 @@ var ScreenShield = class { } this._lockScreenGroup.grab_key_focus(); - - if (Main.sessionMode.currentMode != 'lock-screen') - Main.sessionMode.pushMode('lock-screen'); } _lockScreenShown(params) { @@ -510,8 +504,6 @@ var ScreenShield = class { _continueDeactivate(animate) { this._hideLockScreen(animate); - if (Main.sessionMode.currentMode == 'lock-screen') - Main.sessionMode.popMode('lock-screen'); if (Main.sessionMode.currentMode == 'unlock-dialog') Main.sessionMode.popMode('unlock-dialog'); @@ -579,8 +571,7 @@ var ScreenShield = class { this.actor.show(); - if (Main.sessionMode.currentMode != 'unlock-dialog' && - Main.sessionMode.currentMode != 'lock-screen') { + if (Main.sessionMode.currentMode != 'unlock-dialog') { this._isGreeter = Main.sessionMode.isGreeter; if (!this._isGreeter) Main.sessionMode.pushMode('unlock-dialog'); diff --git a/js/ui/sessionMode.js b/js/ui/sessionMode.js index d77f95445..1c5298770 100644 --- a/js/ui/sessionMode.js +++ b/js/ui/sessionMode.js @@ -53,19 +53,6 @@ const _modes = { panelStyle: 'login-screen', }, - 'lock-screen': { - isLocked: true, - isGreeter: undefined, - unlockDialog: undefined, - components: ['polkitAgent', 'telepathyClient'], - panel: { - left: [], - center: [], - right: ['aggregateMenu'], - }, - panelStyle: 'lock-screen', - }, - 'unlock-dialog': { isLocked: true, unlockDialog: undefined,