Review of "O-tiling" version 2.8.11 (5)

Details Page Preview

Auto-tiling extension for GNOME Shell that automatically arranges your windows into clean, efficient layouts, no manual dragging or resizing required. Built for a fast, efficient workflow, intelligently adapts your layout as windows open, close, and change, while giving you the customization and control you need. Make multitasking on GNOME faster, cleaner, and more productive.

Extension Homepage
https://github.com/oliwebd/o-tiling

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 2

Shexli found 2 issues that may need reviewer attention.

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

File Operations

  • ui/theme_consistency/apply.js:26
    GLib.file_get_contents(path)

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

  • floating_exceptions/config.js
  • floating_exceptions/main.js
  • floating_exceptions/utils.js

All Versions

Version Status
2.10.10 (28) Active
2.10.9 (27) Active
2.10.8 (26) Rejected
2.10.7 (25) Active
2.10.7 (24) Rejected
2.10.4 (23) Active
2.10.3 (22) Rejected
2.10.1 (21) Inactive
2.9.21 (20) Inactive
2.9.20 (19) Rejected
2.9.20 (18) Rejected
2.9.18 (17) Rejected
2.9.17 (16) Rejected
2.9.15 (15) Inactive
2.9.12 (14) Active
2.9.9 (13) Rejected
2.9.5 (12) Inactive
2.9.4 (11) Rejected
2.9.4 (10) Rejected
2.9.3 (9) Rejected
2.9.1 (8) Rejected
2.9.0 (7) Rejected
2.8.13 (6) Inactive
2.8.11 (5) Rejected
2.5.0 (4) Rejected
2.5.0 (3) Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Unnecessary try-catch wrappers in `system/executor.js`. 2. If you want to override the property in line 81 `system/executor.js`, you should remove the older source before that line. 3. Timeout should be removed on disable and before creating a new one (line 194 `ui/panel_settings.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 4. Why try-catch when you used `=== 'function'` (`utils/utils.js`)? Please avoid sending large diff. It is hard to review and takes so much time time to review. You can also add `donations` to the `metadata.json`, so people can donate to you if they want: [Extension Anatomy: donations](https://gjs.guide/extensions/overview/anatomy.html#donations) If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions