-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129212/#review100115
-----------------------------------------------------------



Looks good to me, I'd "Ship It" it but since there's lots of people in the list 
of CC, let's let them give their opinion :)

- Albert Astals Cid


On Oct. 18, 2016, 6:40 a.m., Jan Grulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129212/
> -----------------------------------------------------------
> 
> (Updated Oct. 18, 2016, 6:40 a.m.)
> 
> 
> Review request for Network Management, KDE Usability, Albert Astals Cid, and 
> Lamarque Souza.
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> -------
> 
> As discussed over email, currently the password options (Save, Not Required, 
> Always ask) are not easy to find so we decided to move them back to separated 
> combobox again, ecept moving the combobox below the password field with a 
> better description.
> 
> ![Example of wireless 
> setting](https://jgrulich.fedorapeople.org/password-field-new.png)
> 
> 
> Diffs
> -----
> 
>   libs/editor/settings/cdmawidget.cpp 309b6e6 
>   libs/editor/settings/gsmwidget.cpp de5a7ef 
>   libs/editor/settings/pppoewidget.cpp 1305797 
>   libs/editor/settings/security802-1x.cpp e5f0307 
>   libs/editor/settings/wifisecurity.cpp fc607eb 
>   libs/editor/widgets/passwordfield.h 627d9b0 
>   libs/editor/widgets/passwordfield.cpp 68aa3b8 
>   vpn/openvpn/openvpnadvancedwidget.cpp 1dea814 
>   vpn/openvpn/openvpnwidget.cpp 5241883 
>   vpn/pptp/pptpwidget.cpp dabaa25 
>   vpn/sstp/sstpwidget.cpp f0bf10a 
>   vpn/vpnc/vpncwidget.cpp 7c547f9 
> 
> Diff: https://git.reviewboard.kde.org/r/129212/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

Reply via email to