Review of "Dynamic Brightness" version 1

Details Page Preview

Adjusts your screen brightness to what's on screen: dimmer for bright pages, brighter for dark ones. Easier on the eyes when you switch between light and dark content. Features: • Brightness range right in Quick Settings • Customizable response curve • Smooth transitions • Pauses for fullscreen games and videos Light on resources: the screen is measured only when its content changes, from a tiny thumbnail rendered on the GPU. Notes: • Only the primary monitor is measured; multi-monitor support may come later. • GNOME 49 and later: works together with Automatic Screen Brightness. • GNOME 46–48: turn off Automatic Screen Brightness first, since both control the same setting. Bug reports and feature requests are welcome on GitHub.

Extension Homepage
https://github.com/mnural/dynamic-brightness-gnome

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) warning 2

Shexli found 2 issues that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 9 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:133
    console.log("[DEBUG] Dynamic Brightness extension loaded (Standby mode).")
  • extension.js:145
    console.log("[DEBUG] Dynamic Brightness Disabled.")
  • extension.js:159
    console.log("[DEBUG] Dynamic Brightness loop started by user.")
  • extension.js:169
    console.log("[DEBUG] Dynamic Brightness loop stopped by user.")
  • extension.js:283
    console.log(`[DEBUG] Brightness adjusted! New Target: ${targetBrightness}% (Luminance: ${luminanceRatio.toFixed(2)}%)`)
  • extension.js:208
    console.error(`[ERROR] Silent content capture error: ${e}`)
  • extension.js:253
    console.error(`[ERROR] Async processing error: ${err}`)
  • extension.js:262
    console.error(`[ERROR] Image stream error: ${err}`)
  • extension.js:306
    console.error(`[ERROR] Native DBus error: ${e}`)

EGO-L-003 warning

signals connected by extension should be disconnected in disable()

Signals assigned in `enable()` are missing matching disconnect calls in `disable()` or its helper methods.

Disconnect all signals

  • extension.js:112
                this.toggle.connect('notify::checked', () => {
                    this._indicator.visible = this.toggle.checked;
                })

All Versions

Version Status
10 Active
9 Active
8 Rejected
7 Rejected
6 Inactive
5 Inactive
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

Nural auto- rejected
Auto-rejected because of new version 2 was uploaded