Review of "Snowy" version 5

Details Page Preview

Let it snow! - GNOME 47 support - Multiple monitors support - GUI preferences - Animation configuration - Fully customizable appearance

Extension Homepage
https://github.com/ExposedCat/snowy

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

Artem Prokop posted a review
@JustPerfection GLib.SOURCE_REMOVE does the thing, doesn't it?
Artem Prokop auto- active
JustPerfection posted a review
Not going to reject this one since the code was there from the old versions but you should remove the timeout in disable (extension.js line 72): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources
JustPerfection posted a review
As mentioned in the review guidelines: > You MUST remove all active main loop sources in disable(), even if the callback function will eventually return false or GLib.SOURCE_REMOVE. It's because the timeout can be called in lock screen when the extension is disabled (in racy situations or in case we have unwanted additional delay).