Review of "Start Overlay in Application View" version 1

Details Page Preview

When activating overview (Super button), the application view is shown instead of the view with the windows.

Extension Homepage
https://github.com/Hexcz/Start-Overlay-in-Application-View-for-Gnome-40-

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

Previous Reviews on this Version

JustPerfection waiting for author
Why don't you remove key with `removeKeybinding()` in disable?
Hex_cz posted a review
Hi JustPerfection, how are you? Hope you're doing great. Tbh, as i said on the extension description, i cloned this extension (https://extensions.gnome.org//extension/1198/start-overlay-in-application-view/) from user tmk (https://extensions.gnome.org/accounts/profile/tmk) since for me it's a must-have and just added the 2 necessary changes to make this work again on gnome 40+, these changes are on line 49, show() needed different parameters to call showApps() as i could check on GNOME/gnome-shell/overview.js line 605 and 606, and consequently i needed also to import OverviewControls, as you can see on line 6. The remaining lines are from the original extension.js. I'm sorry i couldn't be more helpful since i'm still a student. I'd love to contribute with other extensions later, but i'm still lacking knowledge about gnome-shell, so i'm still studying while i have some time to kill, then i'm afraid i can't answer your question since i didn't touched disable function and it's entirely from the original extension.js, and i still can't completely understand how this function works. Sorry.
JustPerfection rejected
Hi, That's an old extension. I reject this one but please fix it and send it again. 1. Let me write this one for you. Remove everything in `extension.js` and only do this: ```js const Overview = imports.ui.overview; let originalToggle; function init() { originalToggle = Overview.Overview.prototype.toggle; } function enable() { Overview.Overview.prototype.toggle = function () { if (this.isDummy) return; if (this._visible) this.hide(); else this.showApps(); }; } function disable() { Overview.Overview.prototype.toggle = originalToggle; } ``` 2. No need to mention about a fork in description since the code I gave you is not from theirs. 3. Add GitHub or GitLab link in metadata.json if this package have one: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed 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
Hex_cz posted a review
Hi again! Thanks A LOT!!! Since it's past midnight here i'll do it later when i wake up, but thanks again for the huge help you provided :) I'll study even more hoping i can help in future. At least lemme mention your help on description or github since i have this extension on there.
JustPerfection posted a review
Thanks! You can add my name to the git if you want.