Review of "Grand Theft Focus" version 1

Details Page Preview

Removes the 'Window is ready' notification and brings the window into focus instead.

Extension Homepage
https://github.com/zalckos/GrandTheftFocus

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

Previous Reviews on this Version

JustPerfection waiting for author
You want to change the `uuid`? I'm asking that since you have the same extension here: https://extensions.gnome.org/review/35476
JustPerfection waiting for author
btw, what about removing `Lang` that I mentioned in other extension?
zalckos posted a review
Hi, thank you for taking the time to review my extension. Yes, I'd like to change the uuid since the previous one contained caps. When I remove "Lang" I get the error that "Lang is not defined".
JustPerfection waiting for author
Hi, 1. Remove line 4 and use this in line 13 instead: ```js this._handlerid = global.display.connect('window-demands-attention', this._onWindowDemandsAttention.bind(this)); ``` 2. Also do this in disable: ```js stealmyfocus = null; ``` 3. Also remove 3.x version from metadata.json if you didn't test it in those versions. You wanna fix them and send it again? 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
zalckos posted a review
Thank you for your directions, much appreciated! I basically only slightly altered the code of https://extensions.gnome.org/extension/234/steal-my-focus/ which used to work in those versions but I didn't test the current code so I'll change the metadata too.
JustPerfection rejected
Thanks! Newer version approved.