Review "ApexShot GNOME Integration" version 1

Review of "ApexShot GNOME Integration" version 1

Enhances ApexShot with GNOME Shell integration for window stacking, recording masks, and runtime overlays

No comments.

You can upload the next version by the same form you've posted this version.
You can change the extension icon by clicking on the icon in your extension page.
In case you want to inactivate an approved version, click on inactivate link in your extension page.
You can find developer guides in gjs.guide.
If you need any help with your extension, you can ask us on GNOME Matrix Channel.

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

Shexli found 4 issues that may need reviewer attention.

EGO-X-003 warning

extensions should not call run_dispose in extension code

Extension code should not call `run_dispose()`.

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

EGO-L-001 warning

extension must not create GObject instances or modify shell before enable()

Resource creation or signal/source setup was found outside `enable()`.

extension.js:769

this._previewHelper = new PreviewStackingHelper()

extension.js:770

this._maskService = new RecordingMaskService()

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`.

extension.js:770

this._maskService = new RecordingMaskService()

extension.js:769

this._previewHelper = new PreviewStackingHelper()

Version	Status
1	Unreviewed