Review of "Workspaces Organizer" version 1

Details Page Preview

Horizontal workspaces indicator, shows opened apps icons in each workspace, and give you the ability to switch to another workspace by just scrolling over it and move opened windows to a another workspace buy just dragging them to that workspace. Fork of Workspace Indicator by fmuellner. To check for updates and report issues, and see the doccumentation, please visit the extension's GitHub page: https://github.com/giantturtle/workspaces-organizer-workspaces-organizer.giantturtle.github.com

Extension Homepage
https://github.com/giantturtle/workspaces-organizer-workspaces-organizer.giantturtle.github.com

No comments.

FAQ

Files

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

All Versions

Version Status
5 Active
4 Active
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection waiting for author
Bad upload? Asking that because you have another extension for review: https://extensions.gnome.org/review/66156
markomisk posted a review
This is the good one, the other one is bad ( https://extensions.gnome.org/review/66156 ) but I couldn't find the option to unpublish it.
JustPerfection posted a review
Request for rejection under that extension then.
markomisk posted a review
Ok, I posted a request for rejection under that extension
markomisk posted a review
Added icon fallback when there is no available icon to show in workspace indicator
JustPerfection rejected
1. Remove `settings-schema` from `metadata.json` since you are not using it. 2. Remove gettext imports and import it as explained in the docs (line 19-22 `extension.js` and line 9-12 `prefs.js`): - [Port Guide 45: In Extension](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#extensionutils) - [Port Guide 45: In Prefs](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#preferences) 3. Remove line 46 `extension.js` and rename line 76 to `destroy()`. Then add `super.destroy()` at the end of that line. Also do that for other classes. 4. Timeout should be removed on destroy also before creating a new one (line 159 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 5. `super.destroy()` is the correct call (line 288 `extension.js`). 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