GNOME Shell Extensions

Review of "Clipboard Indicator" version 48

Details Page Preview

Clipboard Indicator

by Tudmotu

The most popular clipboard manager for GNOME, with over 1M downloads

Extension Homepage
https://github.com/Tudmotu/gnome-shell-extension-clipboard-indicator
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
65 Active
64 Active
63 Active
62 Active
61 Active
60 Active
59 Active
58 Inactive
57 Active
56 Active
55 Active
54 Active
53 Active
52 Active
51 Active
50 Active
49 Rejected
48 Rejected
47 Active
46 Rejected
45 Active
44 Active
43 Active
42 Active
41 Rejected
40 Rejected
39 Inactive
38 Active
37 Active
36 Active
35 Inactive
34 Active
33 Active
32 Active
31 Inactive
30 Active
29 Active
28 Active
27 Active
26 Active
25 Active
24 Active
23 Active
22 Active
21 Active
20 Active
19 Active
18 Active
17 Active
16 Active
15 Active
14 Active
13 Active
12 Rejected
11 Active
10 Active
9 Active
8 Active
7 Active
6 Active
5 Active
4 Rejected
3 Rejected
2 Active
1 Rejected

Previous Reviews on this Version

Tudmotu auto- rejected
Auto-rejected because of new version 49 was uploaded
JustPerfection rejected
1. Please remove line 10 prefs.js. 2. Where is `constants.js` (line 8 prefs.js, line 15 extension.js)? 3. Please remove `utils.js` since you are not using it: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files 4. You need to null out `_openDialog` in disable (line 10 confirmDialog.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects 5. Move to enable and null out on disable (line 17 extension.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-init-for-initialization 6. Remove timeout on destroy (line 141, 182 extension.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources 7. Where do you set interval for `this._historyLabelTimeoutId` line 891 (extension.js)?
Tudmotu posted a review
1. Done 2. Added 3. Done 4. Refactored into class 5. Moved to be _init argument 6. Done 7. Lmao no idea, removed Uploaded new version