Review of "Saved Desks" version 10

Details Page Preview

Save and restore window layouts on workspaces (Requires GNOME set to 'Workspaces on primary display only').

Extension Homepage
https://github.com/loginone33/saved-desks

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli did not find any issues.

All Versions

Previous Reviews on this Version

JustPerfection rejected
1. For maximize, it can be approved like that but I highly recommend to version check instead of those try-catch wrappers. 2. What's the reason for not doing this instead (line 541-556 `extension.js`)? ```js const ws = window.get_workspace(); let workArea = ws.get_work_area_for_monitor(window.get_monitor()); ``` 3. What's the reason for wrapping `window.tile` with try-catch? That doesn't throw. Please avoid unnecessary try-catch wrappers. Seems like AI code. AI tends to wrap the functions with try-catch since it has no idea about the API.
loginone posted a review
Replaced the try-catch wrappers with explicit version checks. Simplified the work area logic as suggested. Cleaned up all redundant try-catch blocks. and its sunday :D