Review of "Window Control" version 11 (1)

Details Page Preview

List, move and tile windows from a script, a rules file or the keyboard. It works on Wayland, where wmctrl and xdotool reach only XWayland windows, and on X11. * Scripts: list every window with its ID, title, WM class, PID, workspace, monitor and geometry. Activate, move, resize, minimize, maximize, fullscreen, keep above, make sticky or close a window, move it to another workspace or monitor, or wait until a window opens. Use the wctl command-line client or any D-Bus client, such as gdbus. * Rules: a rule in ~/.config/gnome-window-control/rules.json places a matching window when it opens. The extension reads that file and never writes it. * Keyboard: Super+Ctrl and a numpad key tile the focused window on a 4x2 grid. You can rebind or unbind every shortcut in GSettings. No preferences window and no panel icon. When you disable the extension, the D-Bus interface, the rules and the shortcuts stop. IMPORTANT: the D-Bus interface has no access control. Any application in your session can read your window titles and control your windows. Do not enable the extension if that is not acceptable to you. wctl is a separate download and is not installed with this extension. Get it and the full documentation from the project page.

Extension Homepage
https://github.com/carlo9890/gnome-window-control

No comments.

FAQ

Files

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

Shexli (experimental) error 2 warning 2

Shexli found 4 issues that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 21 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:264
    console.error(`[Window Control] ${label}() error: ${e.message}`)
  • extension.js:354
    console.error(`[Window Control] ${label}() error: ${e.message}`)
  • extension.js:377
    console.error(`[Window Control] ${label} error: ${e.message}`)
  • extension.js:406
    console.error(`[Window Control] List() error: ${e.message}`)
  • extension.js:461
    console.error(`[Window Control] ListDetailed() error: ${e.message}`)
  • extension.js:493
    console.error(`[Window Control] ListMonitors() error: ${e.message}`)
  • extension.js:551
    console.error(`[Window Control] GetFocused() error: ${e.message}`)
  • extension.js:613
    console.error(`[Window Control] GetGeometry() error: ${e.message}`)
  • extension.js:652
    console.error(`[Window Control] GetWorkarea() error: ${e.message}`)
  • extension.js:692
    console.error(`[Window Control] ListWorkspaces() error: ${e.message}`)

EGO-C49-003 error

extensions targeting GNOME 49 must not call maximize or unmaximize with Meta.MaximizeFlags

This extension explicitly targets GNOME Shell 49 but still passes `Meta.MaximizeFlags` to `maximize()` or `unmaximize()`.

Meta.Window

  • extension.js:89
    win.maximize(Meta.MaximizeFlags.BOTH)
  • extension.js:97
    win.unmaximize(Meta.MaximizeFlags.BOTH)

EGO-C49-004 error

extensions targeting GNOME 49 must not call Meta.Window.get_maximized

This extension explicitly targets GNOME Shell 49 but still calls removed `Meta.Window.get_maximized()`.

Meta.Window

  • extension.js:78
    win.get_maximized()

EGO-L-005 warning

owned object references should be released in disable()

Owned references that are cleaned up in `disable()` should also be released with `null` or `undefined`.

Destroy all objects

  • extension.js:123
            this._dbusImpl = Gio.DBusExportedObject.wrapJSObject(
                DBUS_INTERFACE_XML,
                this
            )

All Versions

Version Status
19 (7) Active
18 (6) Rejected
17 (5) Rejected
16 (4) Active
15 (3) Rejected
14 (2) Rejected
11 (1) Active

Previous Reviews on this Version

JustPerfection active
If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions