Review of "Applications Overview Tooltip" version 18

Details Page Preview

Shows a tooltip over applications icons on applications overview with application name and/or description.

Extension Homepage
https://github.com/RaphaelRochet/applications-overview-tooltip

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
22 Active
21 Active
20 Rejected
19 Active
18 Rejected
17 Active
16 Active
15 Active
14 Active
13 Active
12 Active
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Active
3 Active
2 Active
1 Active

Previous Reviews on this Version

JustPerfection rejected
1. Please remove `String.prototype.format` (line 30-32 `extension.js`). Shell already doing that, not needed. 2. Unused (line 1, 9, 17 `extension.js`). 3. Also null out everything you are creating as `this.` in `extension.js` on `disable()`: ```js this._ttbox = null; this._ttlayout = null; this._ttlabel = null; this._ttdetail = null; ``` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 4. Timeout should be removed on disable (line 251 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources)