Review of "Per-Monitor Screen Blank" version 7

Details Page Preview

Blank individual monitors after pointer idle time.

Extension Homepage
https://github.com/nikitakarpei/per-monitor-screen-blank

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) error 2 warning 1

Shexli found 3 issues that may need reviewer attention.

EGO-P-002 error

GSettings schema path must use /org/gnome/shell/extensions base

GSettings schema path must start with `/org/gnome/shell/extensions`.

GSettings Schemas

  • schemas/org.gnome.shell.extensions.per-monitor-screen-blank.gschema.xml
    id='org.gnome.shell.extensions.per-monitor-screen-blank.profile' path=''

EGO-P-004 error

GSettings schema XML filename must match schema id

GSettings schema filename must match `<schema-id>.gschema.xml`.

GSettings Schemas

  • schemas/org.gnome.shell.extensions.per-monitor-screen-blank.gschema.xml
    id='org.gnome.shell.extensions.per-monitor-screen-blank.profile' path=''

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:2173
    this._indicator = new QuickSettings.SystemIndicator()
  • extension.js:2174
        this._toggle = new QuickSettings.QuickMenuToggle({
          iconName: "display-symbolic",
          title: "Screen Blank"
        })

All Versions

Version Status
8 Unreviewed
7 Rejected
6 Unreviewed
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

Nikita Karpei auto- rejected
Auto-rejected because of new version 8 was uploaded