Review of "Browser Switcher" version 1

Details Page Preview

Quick default browser switching from the system panel. Perfect for users working with different browser profiles.

Extension Homepage
https://github.com/totoshko88/browser-switcher

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
11 Active
10 Active
9 Active
8 Active
7 Active
6 Rejected
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Timeout should be removed on destroy and also before creating a new one (line 141 `menuBuilder.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 2. Don't use sync spawn since that can freeze the shell process (line 146 `browserManager.js`): [GJS Guide: Subprocesses](https://gjs.guide/guides/gio/subprocesses.html#complete-examples) 3. Please remove line 96-128 `extension.js`. They are unnecessary. Also, remove the try and catch wrapper from enable and disable. Not needed really. If the code errors out, the logs will have the exact backtrace. 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