Review of "Snowy" version 22

Details Page Preview

Let it snow! - GNOME 46 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
Seems that `window` object is not shared between methods, because `window._settings` is `undefined` even after being set in `fillPreferencesWindow`, however `window` exists in all methods. Probably `this.settings` is a better placement than `window._settings`, but not sure if it will require any kind of cleanup on extension disposal
JustPerfection active
Only when you clean that up on window close request: ```js window.connect('close-request', () => { this._settings = null; }); ```