Review of "App Grid On Startup" version 3

Details Page Preview

Makes it possible so GNOME starts from the app grid instead of the activities overview on startup NOTE: On first install, the App Grid will open once. This is expected behavior and can be ignored

Extension Homepage
https://github.com/AndronikosGl/AppGridOnStartup-Gnome-Extention

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 has not produced a result yet.

All Versions

Version Status
9 Active
8 Active
7 Inactive
6 Inactive
5 Rejected
4 Inactive
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Move line 5 `extension.js` inside `_openAppGrid()`: - [EGO Review Guidelines: Initialization](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources) - [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 2. You shouldn't null out the signal id in the callback (line 14 `extension.js`). 3. As mentioned before, this is not going to work on 45 (#3 from version 2 review). Remove 43 and 44, and use this code in `extension.js`: ```js import Gio from 'gi://Gio'; import * as Main from 'resource:///org/gnome/shell/ui/main.js'; export default class AppGridOnStartup { enable() { if (!Main.layoutManager._startingUp) return; this._settings = new Gio.Settings({ schema: 'org.gnome.desktop.interface' }); Main.layoutManager.connectObject('startup-complete', () => { Main.layoutManager.disconnect(signalId); this.#openAppGrid(); }, this); } disable() { this._settings = null; Main.layoutManager.disconnectObject(this); } #openAppGrid() { const prev = this._settings.get_boolean('enable-animations'); this._settings.set_boolean('enable-animations', false); Main.overview.showApps(); this._settings.set_boolean('enable-animations', prev); } } ``` Didn't test but that's how you should port this code to the 45+. Anyway, I just ported your code to 45+. I still don't recommend disabling animation for this.