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


Fix it, then Ship it!





libs/editor/widgets/passwordfield.h (line 45)
<https://git.reviewboard.kde.org/r/129212/#comment67248>

    The parameter name is "enable", not "enabled".



libs/editor/widgets/passwordfield.cpp (line 76)
<https://git.reviewboard.kde.org/r/129212/#comment67246>

    This should be setPasswordModeEnabled(bool enable)



libs/editor/widgets/passwordfield.cpp (line 107)
<https://git.reviewboard.kde.org/r/129212/#comment67247>

    The & goes next the variable name:
    
    const QString &text


- Lamarque Souza


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