brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

extensions-app: Reset switch handle when it becomes insensitive

Browse Source 
Currently the switches handle position reflects the requested
extension state (as in: the user (de)activated the switch),
while the actual extension state is reflected by the underlying
state.

That doesn't work well when the switch is insensitive though (for
example on error), because the desaturation neuters the color
that reflects the state.

Address this by resetting the switch handle to the state when
making it insensitive.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1263


(cherry picked from commit d2583aa47bd716f196bd1d171d9d134e2bed5d80)
This commit is contained in:
Florian Müllner 2020-05-18 12:24:16 +00:00 committed by Georges Basile Stavracas Neto
parent e210d3138b
commit 28407c95b3
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
subprojects/extensions-app/data/ui/extension-row.ui
View File

@ -54,7 +54,7 @@
</object>
</child>
<child>
<object class="GtkSwitch">
<object class="GtkSwitch" id="switch">
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="valign">center</property>

4
subprojects/extensions-app/js/main.js
View File

@ -334,6 +334,7 @@ var ExtensionRow = GObject.registerClass({
'versionLabel',
'authorLabel',
'updatesIcon',
'switch',
'revealButton',
'revealer',
],
@ -453,6 +454,9 @@ var ExtensionRow = GObject.registerClass({
action.set_state(new GLib.Variant('b', state));
action.enabled = this._canToggle();
if (!action.enabled)
this._switch.active = state;
this._updatesIcon.visible = this.hasUpdate;
this._versionLabel.label = this.version.toString();