Review of "Walkpaper2" version 1

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.

FAQ

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
1. Remove the timout in disable (line 22 extension.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources 2. Use a better gettext domain (line 8 prefs.js and 5 metadata.json). 3. Also include schema xml file: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#gsettings-schemas If you need any help with your extension you can ask us on: - [GNOME Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
yattamax posted a review
1) this is need for avoid (orrible) flickering during the rabid change of desktops. So: is not possible to remove 2) in the previous extension is it was like this, so why now request this change? 3) I don't understand :) No, problem if is not possible accept this exetensione, I continue to use in my local PC :) I was hoping to do something pleasant :)
yattamax posted a review
1) this is need for avoid (orrible) flickering during the rabid change of desktops. So: is not possible to remove 2) in the previous extension is it was like this, so why now request this change? 3) I don't understand :) No, problem if is not possible accept this exetensione, I continue to use in my local PC :) I was hoping to do something pleasant :)
JustPerfection posted a review
1. If that timeout is for monitor background change, why not using `changed` signal for `picture-uri` and `color-scheme`? 2. Well, you are using different gettext domain in metadata.json and in prefs file. and it's something generic, not unique to your extension. 3. I see you sent the fix in version 2. So, #3 issue is solved.