This is what I found on symbolic icons
https://wiki.gnome.org/Design/OS/SymbolicIcons

found out that network manager also does the crick of loading all the icons (the system tray and the ones that appear when clicking over the tray to show the available networks) at the same time. Since the dialog is light background and the panel dark, the best solution might be to load them separately and use non-symbolic for the tray and symbolic for the available network list.

Sounds good to me, @hmollercl. Please do work with upstream on this; I certainly want to SRU this into 19.04 when we have a fix (and we might be able to do a 0 day SRU).

submitted issue upstream https://github.com/palinek/nm-tray/issues/39

palinek added a commit to solve the issue and it worked, the commit is:
https://github.com/palinek/nm-tray/commit/ecbe243339cc4a89dbaf3c37362cd0d963e0a22d
since it is over 0.4.2 and we are in 0.4.1 and in sid there is already 0.4.2-1 https://packages.debian.org/sid/nm-tray how should we proceed?

We need to do a merge/sync but which one depends a bit on what we do with the ubuntu2 patch.

EDIT: This comment is before I read the discussion about the posibility of reverting nm-tray editor change

specifically here, remember that we discuss with agaida and palinek and they will release a new version https://github.com/palinek/nm-tray/issues/41 we should work with it.
My doubt here is that we should merge the actual version in sid or we wait until the new release.

On https://phab.lubuntu.me/rNMTRAYPACKAGING1bc4f5406954093d6 I will add the dependeny as soon as I have some time, I tried pas weekend but I found some difference with sid so I wanted to confirm before doing the change.
I think we should add network-manager-gnome as required, and not recommended.

Regardless of what we decide to do with the connection editor change, having a new release with the icons fixed would be really nice. I had forgotten they planned to do a new release. We should be able to just wait for that to happen and then just do the merge/sync, but let's not forget about it.

yup, beggining of august, could we create an alarm?

{C1}

Bumped the upstream issue.

@hmollercl could you please take a look at SRUing this?

you want to put it 19.04? Or we need and SRU for 19.10?
Should we wait to agaida to put it in sid?

Should we wait to agaida to put it in sid?

For Eoan time is getting short, my personal thought is we should not wait for it to land in sid.

ok, so how should I do it.
1.- Create a new ppa from scratch cloning repo, adding /debian/... etc...
2.- Should I use our repository https://phab.lubuntu.me/source/nm-tray/ somehow? I was thinking something like change the watcher...
3.- Other?

nm-tray sbuild gave me a lintian warn because it doesn't have manpage, what should I do?
override?
nm-tray doesn't have one now, nor has a help

Here it is for you to approve
https://phab.lubuntu.me/D26