Review "Frippery Bottom Panel" version 54

Review of "Frippery Bottom Panel" version 54

Details Page Preview

Frippery Bottom Panel

by rmyorston

Add a bottom panel to the shell

Extension Homepage: http://frippery.org/extensions

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
82	Unreviewed
81	Rejected
80	Rejected
79	Active
78	Active
77	Rejected
76	Active
75	Inactive
74	Active
73	Active
72	Active
71	Active
70	Active
69	Active
68	Active
67	Active
66	Active
65	Rejected
64	Active
63	Active
62	Active
61	Active
60	Active
59	Active
58	Active
57	Rejected
56	Rejected
55	Active
54	Rejected
53	Active
52	Active
51	Active
50	Inactive
49	Active
48	Active
47	Active
46	Active
45	Active
44	Active
43	Active
42	Active
41	Active
40	Active
39	Rejected
38	Rejected
37	Active
36	Rejected
35	Active
34	Active
33	Active
32	Active
31	Active
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	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	Rejected
3	Active
2	Active
1	Active

Previous Reviews on this Version

JustPerfection rejected September 24, 2021

Rejected because ExtensionUtils.initTranslations(); should be moved to init() before init() returning: https://wiki.gnome.org/Projects/GnomeShell/Extensions/Review#Only_use_.60init.28.29.60_for_initialization Also you are doing too much in BottomPanelExtension.constructor(). While it can be done in constructor, it's better to create a method named prepare (or whatever you like) and put all of those in there. then call prepare method from enable().