Review of "Snowy" version 20

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

Version Status
24 Active
23 Active
22 Active
21 Active
20 Rejected
19 Rejected
18 Rejected
17 Rejected
16 Active
15 Active
14 Active
13 Rejected
12 Active
11 Inactive
10 Inactive
9 Rejected
8 Rejected
7 Active
6 Inactive
5 Inactive
4 Inactive
3 Rejected
2 Inactive
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. `setInterval()` is the built in `setInterval()` not `this.setInterval()` (line 14 js/utils.js). 2. Line 16 (js/utils.js) should be doing this, because `return` will remove the timeout automatically: ```js this.timerId = 0; ``` 3. Add this before line 14 (js/utils.js): ```js GLib.Source.remove(this.timerId) ``` 4. To avoid warnings, do this in dispose instead: ```js if (this.timerId) { GLib.Source.remove(this.timerId) } ```