Review "Server Status Indicator" version 13

Review of "Server Status Indicator" version 13

Details Page Preview

Server Status Indicator

by CraigFoote

Indicator displaying status of entered web server URLs using emojis.

Extension Homepage: https://github.com/CraigFoote/ca.footeware.gnomeshell.serverstatus.git

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
32	Active
31	Active
30	Active
29	Active
28	Rejected
27	Active
26	Active
25	Active
24	Active
23	Active
22	Active
21	Active
20	Rejected
19	Active
18	Active
17	Active
16	Active
15	Rejected
14	Rejected
13	Rejected
12	Active
11	Active
10	Rejected
9	Active
8	Active
7	Active
6	Rejected
5	Active
4	Active
3	Active
2	Active
1	Rejected

Previous Reviews on this Version

JustPerfection rejected September 11, 2023

1. No need to override the `clearInterval` (line 10 serverStatusPanel.js): https://gjs-docs.gnome.org/gjs/timers.md#clearinterval 2. Also do this in disable: ```js statusPanels = []; ``` 3. Unused (line 14 extension.js). 4. Don't make them global (line 11-13 prefs.js). They won't get garbage collected after prefs window close.