Version: 4.6 (using KDE 4.6.2)
The widget order is:
the checkbox should be on top (or possibly bottom). This makes more sense, as username and password belong together the checkbox should not separate them.. Also, it makes tabbing through widgets easier: Space>Tab>enter Username>Tab>enter Password>Enter could be used to log onto a server.
*** Bug 276046 has been marked as a duplicate of this bug. ***
Not trying to be annoying or something, but for someone who knows some C++ and where the respective files are, it takes only a few minutes to fix this bug - and it has been open for more than two years now.
Created attachment 82763 [details]
I just made a fixed version. I don't really know how you like your patches, so I just provided the changed file. It works fine so far.
Created attachment 82777 [details]
Separates out 'anonymous login' to separate button to make it clearly a separate concept.
May I offer this counter-proposal? It does require a couple of underlying code changes as well but it's very minor and seems to work well here. It splits off the anonymous checkbox completely into a separate option button to make it clear that anonymous mode disables all of the username/password/domain/etc. options.
Created attachment 82778 [details]
Screenshot of dialog with patch applied
Git commit f04f6abf8e556fa98e838becaf5ca19e316b779c by Michael Pyne.
Committed on 14/10/2013 at 19:10.
Pushed by mpyne into branch 'master'.
kdeui: Streamline KPasswordDialog.
A user reported a couple of years ago that the "Anonymous login"
checkbox is badly placed in KPasswordDialog, appearing in between the
user and password fields and sticking out like a sore thumb.
Instead of just moving the checkbox, I took the opportunity to make the
U/I match the underlying code. Since "Anonymous" mode makes the existing
U/I set both username and password fields to be read-only, it's really a
separate authentication scheme and should be treated as such, especially
since it is hard to tell visually that the Anonymous checkbox is
responsible for disabling the line edits above and below it.
The existing code relies somewhat on the widgets that are present in the
U/I file, so instead of trying to merge the new radio buttons with the
existing widgets I separated out the "anonymous" radio button and its
new counterpart, and left the rest of the existing widgets parented
under a hidden widget to minimize code churn and the risk of introducing
Tested with a handy FTP server, and with the included KPasswordDialog
DIGEST:Improvements to KPasswordDialog
M +29 -27 kdeui/dialogs/kpassworddialog.cpp
M +107 -73 kdeui/dialogs/kpassworddialog.ui