Review of "OmniPanel" version 12

Details Page Preview

OmniPanel is a Multi-Monitor advanced window management solution for productivity, featuring configurable zones for windows, smart auto-placement, and automatic tiling. Features: - Top bar on every active screen, including Extensions - Window tiling with Zone designing and auto-placement - Stack specific windows in Zones - Auto-tiling capability without needing to draw zones - Use either the mouse or hotkeys for switching/moving Master your workflow with perfectly organized, highly customizable layouts for uninterrupted multitasking. Includes a feature to dynamically move the GNOME top bar to your active screen. A GNOME native alternative to FancyZones or COSMIC on Pop!_Os Video: https://www.youtube.com/watch?v=c7JNV20OV8k

Extension Homepage
https://github.com/cwittenhttps://extensions.gnome.org/away/https%253A%252F%252Fgithub.com%252Fcwittenberg%252Fomnipanelberg/omnipanel

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)

Shexli did not find any issues.

All Versions

Previous Reviews on this Version

cwittenberg posted a review
- Migrated to native GNOME language get_text (used AI to help do so) I did not know this native feature existed, really cool. - Ensured timer removal where possible - Where possible removed connectObject() and disconnectObject() Thank you in advance.
JustPerfection rejected
1. You are still doing many unnecessary try-catch wrappers. 2. Timeout should be removed on destroy and also before creating a new one (line 319 `layout_storage.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 3. You don't use destroy to cleanup (line 114 `lifecycle.js`). Also, the destroy/cleanup isn't called on the instance (line 84 `tiling_manager.js`). 4. Selective disable is not allowed: - line 118 `panel_mover.js` - line 312 `stack_manager.js` - line 122 `tiling_manager.js` 5. Remove version from the imports: - `prefs_components.js` - `prefs_guide_about.js` - `prefs_hotkeys.js` - `prefs_tiling.js` - `prefs_topbar.js`