Review of "Adaptive Brightness" version 25

Details Page Preview

This extension is a customizable alternative of 'Automatic Screen Brightness' found in GNOME settings. Features: • Smooth transitions that prevent flickering in changing light • Customizable brightness response to ambient light • Automatic keyboard backlight control in low-light conditions Requires Ambient Light Sensor (ALS) hardware support. Check the extension's home page for device compatibility.

Extension Homepage
https://github.com/dmy3k/gnome-adaptive-brightness

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

Shexli found 1 issue that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

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

No excessive logging

  • lib/KeyboardBacklightService.js:61
    console.error('[KeyboardBacklightService] Failed to start:', error)
  • lib/KeyboardBacklightService.js:123
    console.error('[KeyboardBacklightService] Error setting backlight:', error)
  • lib/KeyboardBacklightService.js:138
    console.error('[KeyboardBacklightService] Error disabling backlight:', error)
  • lib/KeyboardBacklightService.js:170
    console.error('[KeyboardBacklightService] Error in idle/active handler:', error)
  • lib/KeyboardBacklightService.js:176
    console.error('[KeyboardBacklightService] Error starting idle monitoring:', error)
  • lib/KeyboardBacklightService.js:193
    console.error('[KeyboardBacklightService] Error handling idle state:', error)
  • lib/KeyboardBacklightService.js:217
    console.error('[KeyboardBacklightService] Error handling active state:', error)
  • lib/KeyboardBacklightService.js:231
    console.error('[KeyboardBacklightService] Error stopping idle monitoring:', error)

All Versions

Previous Reviews on this Version

fmuellner active
Tip: If you add "gettext-domain": "adaptive-brightness@dmy3k.github.io" in metadata.json, you don't have to call initTranslations() yourself (works for both the extension and prefs)