Comment 12 for bug 883443

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

You don't need a patch to nm-applet to do this; it's a modification that can be reverted in a patch that's already included.

I don't think reverting that commit in shell is a good idea, it will mean modifying both gnome-shell to drop that commit, and nm-applet to not start on shell, with the added possible issues from things failing to work in that case. Not sure if the VPNs still depend on nm-applet running; but that used to be an issue. I'm not really prepared to deal with this so late in the cycle.

I'm testing with the nm-applet patch changed to use just the NORMAL priority. Things seem to look reasonable. You risk missing the notification anyway even when it's critical (when it times out or if it's overriden by another critical notification somehow), and applications themselves already track network status. I think it's reasonable to drop the priority.