Review of "Application Tabs" version 1

Details Page Preview

Window tabs for the focused application in the GNOME top bar or on a draggable floating bar. • Panel mode: tabs in the top bar • Standalone mode: floating tab bar (optional hide top bar, slide animation) • Click tab to focus or minimize; close button and right-click window menu • Drag to reorder tabs; add-tab and recent-windows menus • Per-app fixed panel/standalone mode; customizable tab styles Questions and feature requests: repository issue tracker.

Extension Homepage
https://github.com/hhoao/app_tabs

No comments.

FAQ

Files

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

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Previous Reviews on this Version

JustPerfection waiting for author
Is this a fork?
wong_howe posted a review
Did you mean github repository?
JustPerfection waiting for author
No, I mean if this is a modified version of another extension.
wong_howe posted a review
No, I am a novice, and I followed the documentation and wrote it according to the AppMenuButton in the panel.js source code.
wong_howe posted a review
Is there anything to improve or add?
JustPerfection rejected
1. Please add GitHub or GitLab url to the metadata.json: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed 2. Remove line 5 (metdata.json) since you are not using it. 3. Remove unused files and folders: - `connect/helper.js` - `schemas` folder - `stylesheet.css` - `style/helper.js` - `utils/concurrency.js` https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files 4. Use `console.*` instead of log (utils/log.js and extension.js): - console.debug() - console.error() - console.info() - console.log() - console.warn() 5. That's not getting used anywhere (line 26 AppTabs.js). Remove the import (line 13 AppTabs.js) after removing that line. 6. Unused import (line 12 AppTabs.js). 7. Not needed really (line 320 AppTabs.js). You can use your extension uuid for that. Remove `utils/common.js` after that. GNOME Shell 45 port guide is also out: https://gjs.guide/extensions/upgrading/gnome-shell-45.html If you need any help with your extension you can ask us on: - [GNOME Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
wong_howe posted a review
Thanks!