Review of "O-tiling" version 2.9.18 (17)

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 1

Shexli found 1 issue that may need reviewer attention.

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

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
As mentioned before, the `if` condition in line 3190 `extension.js` should be removed since it falls under selective disable.
oliwebd posted a review
Hi, just to confirm before I resubmit — did the selective disable comment on line 3190 refer to this guard in disable()? disable() { if (ext) { //<=== this one ?? ...teardown... ext = null; } } I've removed the if (ext) check so the teardown always runs unconditionally. Wanted to confirm this is the one you meant before I upload the new version.
JustPerfection posted a review
Yes, that one.