Review of "gNordVPN (nordvpn)" version 21

Details Page Preview

gNordVPN puts NordVPN controls directly in the GNOME top bar. It uses the NordVPN CLI under the hood to show live connection state, surface quick actions, and expose the most useful NordVPN settings without leaving the desktop shell. Features: 🖥️ Top-bar control: Connect, disconnect, and inspect NordVPN status from the GNOME panel. 📡 Live status display: See connection state, country, and server details at a glance. 🌍 Fast server selection: Browse and connect by country, city, or server. ⭐ Favorites: Pin commonly used countries, cities, and servers for quicker access. ⚙️ Connection preferences: Configure options such as auto-connect, protocol, and technology from the extension settings. 🧭 Panel customization: Move the indicator to the left, center, or right side of the top bar. 🎨 Style overrides: Complete customization using CSS for the different VPN states.

Extension Homepage
https://github.com/Isopolito/gNordVPN-Local

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

Previous Reviews on this Version

JustPerfection active
`this.` should be used in methods not pure functions. Please fix them for the next version.
Isopolito posted a review
Hi Just Perfection, thanks for reviewing this. There were major refactors in this release, so it's quite possible the 'this.' instances you're referring to are an artifact of that. I've been looking though and don't find any cases where it's being used outside the scope of a class. Do you have any specific examples? Those are bugs that can cause problems so I want to fix them ASAP. There's small follow up set of changes I plan on doing next that I can include this work in. Thanks,
JustPerfection posted a review
Sorry for not mentioning the file. Search for `this.` in prefs.js.
Isopolito posted a review
ahh I see it, thanks for the heads up. I'll get that fixed soon