Review of "Tray Icons: Reloaded" version 34

Details Page Preview

Tray Icons Reloaded is a GNOME Shell extension which bring back Tray Icons to top panel, with additional features. This extension is no longer maintained and will not receive updates or support. Recommended alternative: AppIndicator and KStatusNotifierItem Support

Extension Homepage
https://github.com/martinpl/tray-icons-reloaded

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 3

Shexli found 3 issues that may need reviewer attention.

EGO-X-002 warning

extensions should not use synchronous subprocess APIs in shell code

Shell code should avoid synchronous subprocess APIs like `GLib.spawn_command_line_sync()` and `GLib.spawn_sync()`.

Complete Examples

  • AppManager.js:45
    GLib.spawn_command_line_sync(`/bin/kill ${pid}`)
  • AppManager.js:139
    GLib.spawn_command_line_sync(
    				`/bin/bash -c 'pmap -p ${pid} | grep Qt'`
    			)
  • AppManager.js:159
    GLib.spawn_command_line_sync(
    				`/bin/bash -c "pidof -s ${wmclass}"`
    			)

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-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:69
    this._settings.connect('changed::icon-brightness', this._setIconEffect.bind(this))
  • extension.js:68
    this._settings.connect('changed::icon-contrast', this._setIconEffect.bind(this))
  • extension.js:62
    this._settings.connect('changed::icon-margin-horizontal', this._setIconSize.bind(this))
  • extension.js:63
    this._settings.connect('changed::icon-margin-vertical', this._setIconSize.bind(this))
  • extension.js:65
    this._settings.connect('changed::icon-padding-horizontal', this._setIconSize.bind(this))
  • extension.js:64
    this._settings.connect('changed::icon-padding-vertical', this._setIconSize.bind(this))
  • extension.js:67
    this._settings.connect('changed::icon-saturation', this._setIconEffect.bind(this))
  • extension.js:61
    this._settings.connect('changed::icon-size', this._setIconSize.bind(this))
  • extension.js:66
    this._settings.connect('changed::icons-limit',  this._setIconsLimit.bind(this))
  • extension.js:58
    this._settings.connect('changed::position-weight', this._setTrayArea.bind(this))
  • extension.js:59
    this._settings.connect('changed::tray-margin-left', this._setTrayMargin.bind(this))
  • extension.js:60
    this._settings.connect('changed::tray-margin-right', this._setTrayMargin.bind(this))
  • extension.js:57
    this._settings.connect('changed::tray-position', this._setTrayArea.bind(this))
  • extension.js:15
    this.tray.connect('tray-icon-added', this._onIconAdded.bind(this))
  • extension.js:16
    this.tray.connect('tray-icon-removed', this._onIconRemoved.bind(this))

All Versions

Version Status
34 Active
33 Active
32 Active
31 Active
30 Active
29 Active
28 Rejected
27 Active
26 Active
25 Active
24 Rejected
23 Active
22 Active
21 Active
20 Active
19 Active
18 Rejected
17 Active
16 Active
15 Active
14 Active
13 Inactive
12 Inactive
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Active
3 Active
2 Rejected
1 Rejected

Previous Reviews on this Version

fmuellner active