Review of "Search Light" version 13

Details Page Preview

Take the apps search out of overview.

Extension Homepage
https://github.com/icedman/search-light

No comments.

Diff Against

Files

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

All Versions

Version Status
31 Active
30 Rejected
29 Active
28 Active
27 Active
26 Rejected
25 Rejected
24 Rejected
23 Rejected
22 Active
21 Active
20 Active
19 Active
18 Rejected
17 Active
16 Inactive
15 Active
14 Active
13 Rejected
12 Active
11 Active
10 Active
9 Rejected
8 Active
7 Active
6 Inactive
5 Active
4 Active
3 Inactive
2 Inactive
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Use `const` instead of `var` (line 7 keybinding.js). 2. You cannot create instance of objects in global scope (line 7 preferences/keys.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-init-for-initialization 3. Where do you use: - `effects/color_effect.glsl` - `effects/color_effect.js` - `utils.js` 4. Null out every member of the default class you are exporting in `extension.js`. For example: ```js this._hiTimer = null; this._loTimer = null; this._style = null; // ... ``` The ideal would be to move all of those methods to another class though, and just create an instance of that class inside enable and null that out in disable. so, only that instance needed to get null out on disable.