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.

Shexli (experimental)

Shexli has not produced a result yet.

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.