Review of "Walkpaper2" version 4

Details Page Preview

Set different wallpaper for each workspace by having the wallpaper switch on workspace change. Forked from the BlinkBP'project (https://github.com/BlinkBP/walkpaper) Unlike the previous extension is added a delay during transition. Is need for avoid flickering during switch using shortcuts :)

Extension Homepage
https://github.com/massimiliano-dalcero/walkpaper

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
7 Active
6 Rejected
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
The timeout still hasn't been removed. You should do something like this: ```js let _changeWallpaperTimeout = null; function _changeWallpaper() { _changeWallpaperTimeout = GLib.timeout_add(GLib.PRIORITY_HIGH, 500, () => { _changeWallpaper_delay(); _changeWallpaperTimeout = null; return GLib.SOURCE_REMOVE; }); } function disable() { if (_changeWallpaperTimeout) { GLib.Source.remove(_changeWallpaperTimeout); _changeWallpaperTimeout = null; } // ... } ```
yattamax posted a review
I'll try again