> 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. > > Tomasz Bywalec wrote: > 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.
Hi Jan Apologies for late reply. I've checked this with version 5.5.4, and it looks like the mentioned issue doesn't occur there. The only other GNU/Linux distribution shipped with KDE 4, which I've managed to verify, is ROSA Desktop Fresh KDE R7 - like Slackware(64)-current, it too uses plasma-nm 0.9.3.?, and the same UI issue occurs there. In other words, it looks like there is no need to prepare a similar patch for Plasma 5. On the other hand, the patch attached in this request could potentially benefit users of KDE 4 (I'm not sure if plasma5-nm can be used in KDE 4 ?). - 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 > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
