Review of "O-tiling" version 2.9.9 (13)

Details Page Preview

O-Tiling is a free, open-source auto-tiling extension for GNOME Shell 48, 49, and 50. It automatically organizes your open windows into a clean, tiled layout - no manual dragging.

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.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

oliwebd posted a review
The EGO-P-007 (warning), that i cant find the way to fix. The floating_exceptions/....(js) is launched as a subprocess via gjs module path... it's a standalone GJS app, not a statically-imported module. Shexli cant see the dynamic reference. I think this is a known limitation of static analysis tools against dynamically-spawned modules.
JustPerfection rejected
Selective disable is not allowed: - line 109 `ui/window_animation.js` - line 30 `ui/workspace_animation.js`
JustPerfection posted a review
Don't need to fix shexli false positive errors.