GNOME Shell Extensions

Review of "AppIndicator and KStatusNotifierItem Support" version 65

Details Page Preview

AppIndicator and KStatusNotifierItem Support

by 3v1n0

Adds AppIndicator, KStatusNotifierItem and legacy Tray icons support to the Shell

Extension Homepage
https://github.com/ubuntu/gnome-shell-extension-appindicator
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 1 warning 4

Shexli found 5 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()`.

GObject.Object.run_dispose

  • statusNotifierWatcher.js:315 
    this._dbusImpl.run_dispose()

EGO-P-006 warning

unnecessary build and translation artifacts should not be shipped

Compiled GSettings schemas should not be shipped for 45+ packages.

Don't include unnecessary files

  • schemas/gschemas.compiled 
    schemas/gschemas.compiled

EGO-P-007 warning

JavaScript files should be reachable from extension.js or prefs.js

Some JavaScript files are not reachable from `extension.js` or `prefs.js` imports.

Don't include unnecessary files

  • tools/busAnalyzer.js

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

Only use initialization for static resources

  • extension.js:36 
            this._watchDogId = this._watchDog.connect('vanished',
            () => this._maybeEnableAfterNameAvailable())
  • extension.js:36 
    this._watchDog.connect('vanished',
            () => this._maybeEnableAfterNameAvailable())

EGO-M-004 error

metadata shell-version must only include plausible stable releases and at most one development release

Field `shell-version` contains invalid values, more than one development release, or implausible future releases.

metadata.json must be well-formed

  • metadata.json 
    shell-version=['45', '46', '47', '48', '49', '50', '51']

All Versions

Version Status
65 Rejected
64 Active
63 Active
62 Rejected
61 Active
60 Active
59 Active
58 Active
57 Active
56 Active
55 Rejected
54 Rejected
53 Active
52 Active
51 Active
50 Active
49 Active
48 Active
47 Active
46 Active
45 Active
44 Active
43 Active
42 Active
41 Active
40 Active
39 Rejected
38 Rejected
37 Active
36 Active
35 Active
34 Active
33 Active
32 Active
31 Active
30 Active
29 Active
28 Active
27 Active
26 Active
25 Rejected
24 Active
23 Active
22 Active
21 Active
20 Active
19 Active
18 Active
17 Active
16 Rejected
15 Active
14 Active
13 Active
12 Rejected
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Rejected
3 Rejected
2 Rejected
1 Active

Previous Reviews on this Version

JustPerfection rejected
Please add 51 once the stable version is released. Also, please fix the issues mentioned in the shexli.