Review of "Global Menu for GNOME" version 17

Details Page Preview

Brings a macOS-style global menu bar to GNOME. Adds contextual File, Edit, View, Go, Window, and Help menus for the focused app, plus a configurable System Menu with quick access to Settings, Terminal, Software Center, Force Quit, and power options. Create your own custom menus with shell commands or keyboard shortcuts, and personalize the System Menu icon from bundled distro logos, an Apple-style icon, or your own image.

Extension Homepage
https://github.com/ShiroOSL/global-menu-for-gnome

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 manual_review 1

Shexli found 4 issues that may need reviewer attention.

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • clipboardHistory.js:40
    St.Clipboard.get_default()
  • clipboardHistory.js:127
    St.Clipboard.get_default()
  • emojiPicker.js:2117
    St.Clipboard.get_default()

EGO-P-006 warning

unnecessary build and translation artifacts should not be shipped

Package contains files that often should not be shipped for review.

Don't include unnecessary files

  • install.sh
    install.sh

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

  • systemInfoPanel.js

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

  • menuManager.js:365
                menuItem.connect("activate", () => {
                  if (this._clipboardPanel) this._clipboardPanel.toggle(this);
                })
  • menuManager.js:369
                menuItem.connect("activate", () => {
                  if (this._emojiPicker) this._emojiPicker.toggle(this);
                })
  • menuManager.js:373
                menuItem.connect("activate", () => {
                  this._executeNativeAction(item.action);
                })

All Versions

Version Status
20 Unreviewed
19 Rejected
18 Rejected
17 Rejected
16 Rejected
15 Active
14 Active
13 Rejected
12 Active
11 Rejected
10 Rejected
9 Active
8 Rejected
7 Active
6 Active
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

Shiro auto- rejected
Auto-rejected because of new version 18 was uploaded