Review of "Retro LCD 7-Segment Clock" version 44

Details Page Preview

Vintage digital display with high-fidelity 7-segment font. Features Neon Green, Amber, and Retro Gray modes with integrated alarm.

Extension Homepage
https://gitlab.com/corral1976/gnome-shell-extension-relojlcd

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 did not find any issues.

All Versions

Previous Reviews on this Version

Corral1976 posted a review
Version 44 Changes New Feature: Flicker Effect Added a subtle vintage LCD flicker effect that simulates the natural flickering of old digital displays. This feature: Works with all 8 color themes (Green, Amber, Gray, Ruby, Sapphire, White, Violet, Gold) Can be toggled on/off from the preferences panel Uses random opacity variations to create an authentic vintage feel Automatically disables during alarm and re-enables when alarm stops Optimized with proper timeout management to prevent memory leaks Adjusted opacity ranges for White and Gray themes to ensure visibility Improvement: Widget Drag Behavior Improved the desktop widget drag interaction to use traditional drag-and-drop behavior: Now requires holding the left mouse button while dragging (more intuitive) Widget settles automatically when button is released Replaced inputGuard with Clutter.Grab for better event handling Added proper signal cleanup in disable() to prevent memory leaks Both changes maintain backward compatibility and improve user experience while keeping the extension lightweight and efficient.
JustPerfection rejected
The way you handle the timeout in line 257 and 264 `extension.js` is a bad practice. If you want to have two timeouts with the same property name, you should create a helper function that creates the timeout in it. So, you can easily remove the older timeout id in that helper function.