Review of "ColorTint" version 5

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 rejected
1. Remove logs. End users shouldn't see that much logs. 2. Remove Extension class and returning Extension class in init() function (extension.js). 3. Fix these warnings (Tested on GNOME 40.1 Fedora 35): May 15 08:03:51 fedora gnome-shell[1751]: Usage of object.actor is deprecated for MenuButton get@resource:///org/gnome/shell/ui/environment.js:317:29 _init@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:142:9 wrapper@resource:///org/gnome/gjs/modules/script/_legacy.js:83:27 enable@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:254:12 _callExtensionEnable@resource:///org/gnome/shell/ui/extensionSystem.js:167:32 _onEnabledExtensionsChanged/<@resource:///org/gnome/shell/ui/extensionSystem.js:493:35 _onEnabledExtensionsChanged@resource:///org/gnome/shell/ui/extensionSystem.js:493:14 createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:516:46 enableExtension@resource:///org/gnome/shell/ui/extensionSystem.js:194:29 EnableExtension@resource:///org/gnome/shell/ui/shellDBus.js:310:38 _handleMethodCall@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:310:38 _wrapJSObject/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:387:34 May 15 08:03:51 fedora gnome-shell[1751]: Usage of object.actor is deprecated for Slider get@resource:///org/gnome/shell/ui/environment.js:317:29 _init@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:187:9 wrapper@resource:///org/gnome/gjs/modules/script/_legacy.js:83:27 enable@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:254:12 _callExtensionEnable@resource:///org/gnome/shell/ui/extensionSystem.js:167:32 _onEnabledExtensionsChanged/<@resource:///org/gnome/shell/ui/extensionSystem.js:493:35 _onEnabledExtensionsChanged@resource:///org/gnome/shell/ui/extensionSystem.js:493:14 createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:516:46 enableExtension@resource:///org/gnome/shell/ui/extensionSystem.js:194:29 EnableExtension@resource:///org/gnome/shell/ui/shellDBus.js:310:38 _handleMethodCall@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:310:38 _wrapJSObject/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:387:34 May 15 08:03:51 fedora gnome-shell[1751]: Usage of object.actor is deprecated for Slider get@resource:///org/gnome/shell/ui/environment.js:317:29 _init@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:194:9 wrapper@resource:///org/gnome/gjs/modules/script/_legacy.js:83:27 enable@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:254:12 _callExtensionEnable@resource:///org/gnome/shell/ui/extensionSystem.js:167:32 _onEnabledExtensionsChanged/<@resource:///org/gnome/shell/ui/extensionSystem.js:493:35 _onEnabledExtensionsChanged@resource:///org/gnome/shell/ui/extensionSystem.js:493:14 createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:516:46 enableExtension@resource:///org/gnome/shell/ui/extensionSystem.js:194:29 EnableExtension@resource:///org/gnome/shell/ui/shellDBus.js:310:38 _handleMethodCall@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:310:38 _wrapJSObject/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:387:34 May 15 08:03:51 fedora gnome-shell[1751]: Usage of object.actor is deprecated for Slider get@resource:///org/gnome/shell/ui/environment.js:317:29 _init@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:201:9 wrapper@resource:///org/gnome/gjs/modules/script/_legacy.js:83:27 enable@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:254:12 _callExtensionEnable@resource:///org/gnome/shell/ui/extensionSystem.js:167:32 _onEnabledExtensionsChanged/<@resource:///org/gnome/shell/ui/extensionSystem.js:493:35 _onEnabledExtensionsChanged@resource:///org/gnome/shell/ui/extensionSystem.js:493:14 createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:516:46 enableExtension@resource:///org/gnome/shell/ui/extensionSystem.js:194:29 EnableExtension@resource:///org/gnome/shell/ui/shellDBus.js:310:38 _handleMethodCall@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:310:38 _wrapJSObject/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:387:34 May 15 08:03:51 fedora gnome-shell[1751]: Usage of object.actor is deprecated for Slider get@resource:///org/gnome/shell/ui/environment.js:317:29 _init@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:209:9 wrapper@resource:///org/gnome/gjs/modules/script/_legacy.js:83:27 enable@/home/ice/.local/share/gnome-shell/extensions/colortint@matt.serverus.co.uk/extension.js:254:12 _callExtensionEnable@resource:///org/gnome/shell/ui/extensionSystem.js:167:32 _onEnabledExtensionsChanged/<@resource:///org/gnome/shell/ui/extensionSystem.js:493:35 _onEnabledExtensionsChanged@resource:///org/gnome/shell/ui/extensionSystem.js:493:14 createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:516:46 enableExtension@resource:///org/gnome/shell/ui/extensionSystem.js:194:29 EnableExtension@resource:///org/gnome/shell/ui/shellDBus.js:310:38 _handleMethodCall@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:310:38 _wrapJSObject/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:387:34 Disabling extension Error: Attempting to remove actor of type 'StBin' from group of class 'Gjs_ui_layout_UiActor', but the container is not the actor's parent.