Review Request 127172: PPTP authentication dialog: focus is now automatically set on "password" textbox.

[Tomasz Bywalec]

> On Feb. 25, 2016, 6:18 a.m., Jan Grulich wrote:
> > Looks good, but branch 0.9.3 is already unmaintained. Is the same patch necessary in Plasma 5? I don't remember having this problem with OpenVPN/VPNC password dialogs and not sure right now if PPTP is different in this.

Hi Jan

I've checked this patch only with version 0.9.3.6, because this is the version of plasma-nm which is shipped with the latest version of Slackware64-current, which I use on an everyday basis. I will try to check this with version 5 and will get back to you.


- Tomasz


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


On Feb. 24, 2016, 11:05 p.m., Tomasz Bywalec wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127172/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2016, 11:05 p.m.)
> 
> 
> Review request for Network Management.
> 
> 
> Repository: plasma-nm
> 
> 
> Description
> -------
> 
> PPTP authentication dialog: focus is now automatically set on "password" textbox.
> 
> Also:
>     
> * The dialog's width is now the same as width of OpenVPN authentication dialog.
> * "Show password" checkbox is now aligned to the left.
> 
> =================================================================================
> 
> Motivation for this change:
> 
> 1. User experience: when keyboard focus is automatically set on "password" textbox, then user can start entering password immediately when the dialog is displayed.
> 
> 2. Consistency of look&behaviour between PPTP and OpenVPN authentication dialogs.
> 
> 
> Diffs
> -----
> 
>   vpn/pptp/pptpauth.cpp a2b1255 
>   vpn/pptp/pptpauth.ui 86bbdb8 
> 
> Diff: https://git.reviewboard.kde.org/r/127172/diff/
> 
> 
> Testing
> -------
> 
> Tested on Slackware64-current.
> 
> 
> Thanks,
> 
> Tomasz Bywalec
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-networkmanager/attachments/20160225/ce68a18e/attachment.html>