Review of "Context + Window Title" version 50.4 (6)

Details Page Preview

Add a little more text and context to your user experience. Creates a top bar “context” button with the focused window’s icon and title, or an app grid icon if no window is focused. Optionally shows a more customizable clock, the user/host name on the system menu button, and a message on the lock screen. The context button can function both as a toggle from desktop to apps, windows, and back to desktop when clicked, and as an app menu when activated using the secondary mouse button, long-press, or a keyboard shortcut.

Extension Homepage
https://github.com/erikis/context-window-title

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

Version Status
50.6 (11) Active
50.5 (10) Active
50.4.3 (9) Active
50.4.2 (8) Active
50.4.1 (7) Rejected
50.4 (6) Rejected
50.3 (5) Rejected
50.2 (4) Rejected
50.1 (3) Rejected
50.1 (2) Rejected
50.0 (1) Rejected

Previous Reviews on this Version

erikis posted a review
Thanks for your valuable feedback. I've tried to fix everything. For 5., different timeouts in the same property in `widgets/clockLabel.js`, I've made it more clear that at any time there is only one timeout and that there are different implementations of it. For 7., disconnect signals on destroy, I've added a missing if statement around the given line, but as for disconnects on destroy, I have previously cleaned those up as only connectObject is used with this as the object. As such, I learned that disconnectObject for those on destroy is automatic and not necessary. However, if you think it is better, I could of course add back the disconnectObject lines.
erikis auto- rejected
Auto-rejected because of new version 50.4.1 (7) was uploaded