Review "Clipboard Indicator" version 48

Review of "Clipboard Indicator" version 48

Details Page Preview

Clipboard Indicator

by Tudmotu

The most popular clipboard manager for GNOME, with over 2M 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.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version	Status
70	Unreviewed
69	Active
68	Active
67	Rejected
66	Active
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 September 12, 2023

Auto-rejected because of new version 49 was uploaded

JustPerfection rejected September 12, 2023

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 September 12, 2023

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