Review of "ColorTint" version 6

Details Page Preview

Tint your desktop with a color of your choice to help with dyslexia, scopic sensitivity, and related conditions.

Extension Homepage
https://github.com/MattByName/color-tint

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Previous Reviews on this Version

JustPerfection active
Approved. Please fix these warnings in the next release. You can see that with `journalctl -f`: May 15 09:06:56 fedora gnome-shell[1751]: Attempting to remove actor of type 'StBin' from group of class 'Gjs_ui_layout_UiActor', but the container is not the actor's parent. May 15 09:06:58 fedora gnome-shell[1751]: Some code called array.toString() on a Uint8Array instance. Previously this would have interpreted the bytes of the array as a string, but that is nonstandard. In the future this will return the bytes as comma-separated digits. For the time being, the old behavior has been preserved, but please fix your code anyway to explicitly call ByteArray.toString(array). (Note that array.toString() may have been called implicitly.) 0 loadColor() ["/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js":95:37] 1 wrapper() ["resource:///org/gnome/gjs/modules/script/_legacy.js":83:26] 2 start_up() ["/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js":110:13] 3 wrapper() ["resource:///org/gnome/gjs/modules/script/_legacy.js":83:26] 4 enable() ["/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js":252:11] 5 _callExtensionEnable() ["resource:///org/gnome/shell/ui/extensionSystem.js":167:31] 6 _onEnabledExtensionsChanged/<() ["resource:///org/gnome/shell/ui/extensionSystem.js":493:34] 7 forEach() ["self-hosted":225:0] 8 _onEnabledExtensionsChanged() ["resource:///org/gnome/shell/ui/extensionSystem.js":493:13] 9 _onEnabledExtensionsChanged() ["self-hosted":850:0] 10 createCheckedMethod/<() ["resource:///org/gnome/gjs/modules/core/overrides/Gio.js":516:45] 11 enableExtension() ["resource:///org/gnome/shell/ui/extensionSystem.js":194:28] 12 EnableExtension() ["resource:///org/gnome/shell/ui/shellDBus.js":310:37] 13 _handleMethodCall() ["resource:///org/gnome/gjs/modules/core/overrides/Gio.js":310:37] 14 _wrapJSObject/<() ["resource:///org/gnome/gjs/modules/core/overrides/Gio.js":387:33]
MatthewBarnard posted a review
Thank you! I'll work on those warning for the next release.