broulik added inline comments.

INLINE COMMENTS

> networkstatus.cpp:226
> +                        case NetworkManager::Portal:
> +                            status = i18n("Connected to %1 (behind captive 
> portal)", connection->name());
> +                            break;

I think this is  bit too technical. Maybe something like "authentication 
required" or whatever we use in the captive portal notification is better 
wording

REPOSITORY
  R116 Plasma Network Management Applet

REVISION DETAIL
  https://phabricator.kde.org/D23852

To: jgrulich, broulik
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to