Review of "WG Indicator" version 1

Details Page Preview

A status indicator for a Wireguard connection.


No comments.

FAQ

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
2 Active
1 Rejected

Previous Reviews on this Version

asterios posted a review
It is a fork of the vpn-indicator-gnome-extension. https://extensions.gnome.org/extension/1134/vpn-indicator/ If your wireguard interface is not named like wg0 just change the grep in line 26 in the extension.js file.
andyholmes waiting for author
This looks mostly good, however you should be destroying the mainloop source in `disable()`, otherwise the loop source, will hold a reference to `twMenu` and the loop will continue to run even if the extension is disabled or uninstalled. Something this should fix it: ``` disable() { if (twMenu && twMenu._timeout) { Mainloop.source_remove(twMenu._timeout); } twMenu.destroy(); twMenu = null; } ```
asterios posted a review
Thank you very much, i will inclide it. -- disable() { if (twMenu && twMenu._timeout) { Mainloop.source_remove(twMenu._timeout); } twMenu.destroy(); twMenu = null; } --
andyholmes rejected
Newer version approved.