Review of "Headphone Volume Limiter" version 1.0.0 (3)

Details Page Preview

Cap media volume when headphones (wired or Bluetooth) are the active output — like the hearing-safety feature on phones. Configurable threshold, three action modes (warn-allow, warn-prompt, block), per-device targeting, and a Quick Settings toggle.

Extension Homepage
https://github.com/imnotndesh/headphone-volume-limiter

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

Version Status
1.0.0 (6) Unreviewed
1.0.0 (5) Rejected
1.0.0 (4) Rejected
1.0.0 (3) Rejected
1.0.0 (2) Rejected
1.0.0 (1) Rejected

Previous Reviews on this Version

Imnotndesh posted a review
Implemented shexli recommended fixes
JustPerfection rejected
1. Please don't use old imports (line 101 `extension.js`). 2. Please use `connectObject()` and `disconnectObject()` so it is easier to track for cleanup (`extension.js` only). 3. Remove unnecessary comments. 4. Use `console.*` instead of `log()`. Logs should be only available on debug mode. For debug use `console.debug()` instead of `console.log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) Also, avoid too much logs. 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