Closed rigelk closed 9 years ago
Hello,
Thanks for that! It looks great. I will check and test in details and include that pull request asap.
Sylvain
On Tue, Nov 3, 2015 at 11:11 AM, Pierre-Antoine Rault < notifications@github.com> wrote:
A detailed list of changes:
- README.md: updated to v3 roadmap. updated line wrap.
- extentsion.js: added imports, notably use of libshell and libsoup. added functions related to ipv6, device rememeberance, public ip display, pref menu opening.
- metadata.json: added schema properties.
- schema/ (new)
- schema/org.gnome.shell.extensions.shew-ip.gschema.xml (new): schema to remember data for the app.
- schema/gschema.compiled (new)
- prefs.js (new): configures the preferences window.
- convenience.js (new): helper file to get settings saved in the schema.
Tested with wired, wireless and virtual interfaces. The only thing still to do before the V2 is to make the pref window change the PopupMenu
without the need of a manual Alt+F2.
You can view, comment on, or merge this pull request online at:
https://github.com/sgaraud/gnome-extension-show-ip/pull/4 Commit Summary
- Added v2 functionnalities : ipv6, device rememberance, public ip, pref menu.
File Changes
- M README.md https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-0 (40)
- A convenience.js https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-1 (93)
- M extension.js https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-2 (276)
- M metadata.json https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-3 (2)
- A prefs.js https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-4 (68)
- A schemas/gschemas.compiled https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-5 (0)
- A schemas/org.gnome.shell.extensions.show-ip.gschema.xml https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-6 (19)
- M show_ip_screenshot.png https://github.com/sgaraud/gnome-extension-show-ip/pull/4/files#diff-7 (0)
Patch Links:
- https://github.com/sgaraud/gnome-extension-show-ip/pull/4.patch
- https://github.com/sgaraud/gnome-extension-show-ip/pull/4.diff
— Reply to this email directly or view it on GitHub https://github.com/sgaraud/gnome-extension-show-ip/pull/4.
Thanks again. I like the way you implemented the features. It gave me an idea of improvement for the public ip display. And I will work on making the preference changes applied immediately.
Thanks @sgaraud Do you have an idea on how to implement that feature? I’ve already looked at the system-monitor-extension pref.js code, so far with no clue though its use shows such a feature at work.
Honestly I have no idea. I will also have to dig into the code of other extensions as good documentation on gjs especially for gnome-shell is not easy to find. plan to spend time on that during the weekend.
A detailed list of changes:
Tested with wired, wireless and virtual interfaces. The only thing still to do before the V2 is to make the pref window change the PopupMenu without the need of a manual
Alt+F2
.