Review of "quake-mode" version 10

Details Page Preview

Drop-down mode for any application

Extension Homepage
https://github.com/repsac-by/gnome-shell-extension-quake-mode

No comments.

Diff Against

Files

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

All Versions

Version Status
10 Rejected
9 Active
8 Active
7 Active
6 Inactive
5 Inactive
4 Inactive
3 Active
2 Inactive
1 Inactive

Previous Reviews on this Version

JustPerfection rejected
1. Bad path and id: [EGO Review Guidelines: GSettings Schemas](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#gsettings-schemas) 2. Timeout should be removed on destroy (line 166 `quakemodeapp.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 3. You cannot import `Gdk` to the GNOME Shell process (line 1 `util.js`): [EGO Review Guidelines: import](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#do-not-import-gtk-libraries-in-gnome-shell) 4. Use `console.*` instead of `log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 5. `lookupByURL()` is a bad practice. You can send `this` from the entry point to the class needing it when it's possible (dependency injection). - line 14 `indicator.js` - line 28 and 293 `prefs.js` - line 40 `quakemodeapp.js` 6. Extensions cannot use `run_dispose()`: - line 90 `extension.js` - line 59 `quakemodeapp.js` [EGO Review Guidelines: run_dispose](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-should-not-force-dispose-a-gobject) 7. Move to enable and null it out in disable (line 23 `extension.js`): [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 8. Extensions must not disable selectively (line 84 `extension.js`): [EGO Review Guidelines: Session Mode](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#session-modes) and `apps` should be cleared on disable no matter on which session is getting disabled. 9. Moving line 181 `extension.js` to disable is much better for readability. 10. Please move to `fillPreferencesWindow()` function: [Preferences Guide: prefs.js](https://gjs.guide/extensions/development/preferences.html#prefs-js)