From 6263d5e115a726dcd75a5368afdd2798236fb1de Mon Sep 17 00:00:00 2001 From: Hans de Goede Date: Wed, 15 Aug 2018 15:03:56 +0200 Subject: [PATCH 2/2] endSessionDialog: Support rebooting into the bootloader menu aka ("Boot Options") This implements the "Alt" behavior for the "Reboot" button as outlined in the design here: https://wiki.gnome.org/Design/OS/BootOptions This causes the endSessionDialog to send a ConfirmedRebootToBootOptions signal to gnome-session instead of the normal ConfirmedReboot signal, actually telling the boot-loader that it should show its menu the next boot is left up to gnome-session. Note I've tried implemeting this with the AltSwitcher class from js/ui/status/system.js first, but that puts the button in a St.Bin() which causes the button to think it is the only button on the dialog and makes it have rounded corners on both of its bottom corners. --- js/ui/endSessionDialog.js | 51 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 51 insertions(+) diff --git a/js/ui/endSessionDialog.js b/js/ui/endSessionDialog.js index 301d8c9bf..0dc717241 100644 --- a/js/ui/endSessionDialog.js +++ b/js/ui/endSessionDialog.js @@ -256,6 +256,9 @@ class EndSessionDialog extends ModalDialog.ModalDialog { this._totalSecondsToStayOpen = 0; this._applications = []; this._sessions = []; + this._capturedEventId = 0; + this._rebootButton = null; + this._rebootButtonAlt = null; this.connect('destroy', this._onDestroy.bind(this)); @@ -439,6 +442,26 @@ class EndSessionDialog extends ModalDialog.ModalDialog { this._sessionHeader.visible = hasSessions; } + _onCapturedEvent(actor, event) { + let altEnabled = false; + + let type = event.type(); + if (type != Clutter.EventType.KEY_PRESS && type != Clutter.EventType.KEY_RELEASE) + return Clutter.EVENT_PROPAGATE; + + let key = event.get_key_symbol(); + if (key != Clutter.KEY_Alt_L && key != Clutter.KEY_Alt_R) + return Clutter.EVENT_PROPAGATE; + + if (type == Clutter.EventType.KEY_PRESS) + altEnabled = true; + + this._rebootButton.visible = !altEnabled; + this._rebootButtonAlt.visible = altEnabled; + + return Clutter.EVENT_PROPAGATE; + } + _updateButtons() { this.clearButtons(); @@ -460,7 +483,33 @@ class EndSessionDialog extends ModalDialog.ModalDialog { }, label }); + + // Add Alt "Boot Options" option to the Reboot button + if (signal == 'ConfirmedReboot') { + this._rebootButton = button; + this._rebootButtonAlt = this.addButton({ + action: () => { + this.close(true); + let signalId = this.connect('closed', () => { + this.disconnect(signalId); + this._confirm('ConfirmedRebootToBootOptions'); + }); + }, + label: C_("button", "Boot Options") + }); + this._rebootButtonAlt.visible = false; + this._capturedEventId = global.stage.connect('captured-event', this._onCapturedEvent.bind(this)); + } + } + } + + _stopAltCapture() { + if (this._capturedEventId > 0) { + global.stage.disconnect(this._capturedEventId); + this._capturedEventId = 0; } + this._rebootButton = null; + this._rebootButtonAlt = null; } close(skipSignal) { @@ -472,6 +521,7 @@ class EndSessionDialog extends ModalDialog.ModalDialog { cancel() { this._stopTimer(); + this._stopAltCapture(); this._dbusImpl.emit_signal('Canceled', null); this.close(); } @@ -480,6 +530,7 @@ class EndSessionDialog extends ModalDialog.ModalDialog { let callback = () => { this._fadeOutDialog(); this._stopTimer(); + this._stopAltCapture(); this._dbusImpl.emit_signal(signal, null); }; -- 2.21.0