Review of "Multi Monitor Bar" version 42

Details Page Preview

Fork of spin83's Multi Monitors. Pleaseeeee donate to support this extension :). Don't forget to star the original repository! , rate and comment

Extension Homepage
https://github.com/FrederykAbryan/multi-monitors-bar_fapv2

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

JustPerfection rejected
1. Please avoid using AI when it generates unnecessary code. All the try-catch wrappers have to go. No really needed to wrap disconnect and source remove with try-catch. checks like line 416 `extension.js` is really not needed. Same issue is happening in other lines. 2. `this._isDestroyed` is a bad practice. The property holding the instance should be null out after destroy so it is not accessed after that. 3. Avoid sending large diff. It's hard to review.