Review of "Display Configuration Switcher" version 13

Details Page Preview

Quickly change the display configuration from the system menu. Multiple display configurations can be saved with custom names. A display configuration contains all display settings: resolution, scaling, orientation, position...

Extension Homepage
https://gitlab.com/knokelmaat/display-configuration-switcher-gnome-extension

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) warning 1

Shexli found 1 issue that may need reviewer attention.

EGO-L-003 warning

signals connected by extension should be disconnected in disable()

Signals assigned in `enable()` are missing matching disconnect calls in `disable()` or its helper methods.

Disconnect all signals

  • extension.js:58
                this._displayConfigSwitcher.connect('state-changed', () => {
                    this._updateMenu();
                })
  • extension.js:70
                this._settings.connect('changed::display-configuration-switcher-shortcuts-enabled', (settings, key) => {
                    this._updateKeyBindings();
                })

All Versions

Version Status
13 Active
12 Active
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Active
3 Active
2 Active
1 Rejected

Previous Reviews on this Version

fmuellner active