<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100920/">http://git.reviewboard.kde.org/r/100920/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 22nd, 2011, 6:23 p.m., <b>John Layt</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">My thoughts are if the anonymous tickbox is first focus it should be above the username/password fields and the username/password fields should be disabled if anonymous is selected. But that may look a little weird with the text to the left and the title below it.
Or does entering a username/password automatically de-select the anonymous? In which case this is fine so long as the username field has the cursor focus and pressing enter immediately activates the ok button.
Or perhaps pre-populate username with 'anonymous' and have the word pre-selected so it can be overtyped and do away with the tickbox entirely?
Basically what I'm looking for is the easiest shortest most obvious path for the main use cases.
</bikeshedding></pre>
</blockquote>
<p>On March 22nd, 2011, 6:56 p.m., <b>Thomas Lübking</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">> the anonymous tickbox is first focus it should be above the username/password
+1
> username/password fields should be disabled if anonymous is selected
the problem is about checking the box will disable the UI.
can we maybe drop the checkbox entirely and just QLinedit::setPlaceholderText("anonymous") and QLinedit::setPlaceholderText("no password required for anonymous login") or sth. like this?
The tab order can btw be controlled explicitly and unrelated to the actual layout.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">> Or does entering a username/password automatically de-select the anonymous? In which case this is fine so long as the username field has the cursor focus and pressing enter
> immediately activates the ok button.
Nope. You have to manually deselect the checkbox to enter the username/password. Otherwise, those input widgets will be set to read only.
> Basically what I'm looking for is the easiest shortest most obvious path for the main use cases.
Unfortunately, that is the problem. This feature does not solve any main use case. Rather it seems to have been added to deal with a very corner case in specific ioslaves, ftp being the main or only?? one.
> can we maybe drop the checkbox entirely and just QLinedit::setPlaceholderText("anonymous") and QLinedit::setPlaceholderText("no password required for anonymous
> login") or sth. like this?
Well, it seems that the anonymous checkbox was specifically added to distinguish between the default "anonymous" login and a user entered "anonymous" user name for the ftp ioslave. At least that is what I get reading the commit message in the ftp ioslave that seems to be related to this checkbox [1]. As such, I fail to see how the placeholder text approach would satisfy what the checkbox is attempting to achieve. However, I personally do not think that checkbox should exist at all. It is there to handle a very corner case for very specific ioslaves (mainly ftp ioslave). Even then it seems to be used incorrectly right now. Anyhow, I like the placeholder idea but I will serve the same purpose as the checkbox.
> The tab order can btw be controlled explicitly and unrelated to the actual layout.
Right, but the checkbox will still look out of place where it is right now.
[1] http://quickgit.kde.org/?p=kdelibs.git&a=commit&h=2e48eecc2fda83f08fb5f55519fe51793f536734</pre>
<br />
<p>- Dawit</p>
<br />
<p>On March 22nd, 2011, 6:04 p.m., Dawit Alemayehu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated March 22, 2011, 6:04 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The attached patch renames and moves the "Anonymous" box shown in the first screenshot below the password input widgets. This change is done to
#1. Remove the checkbox from screwing tab order when you enter username/password.
#2. Make the purpose of the checkbox less cryptic to the end user.
Any objections ?</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kdeui/dialogs/kpassworddialog.ui <span style="color: grey">(2649870)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100920/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/100920/s/106/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/03/22/current_password_dialog_box_400x100.png" style="border: 1px black solid;" alt="Current Password Dialog" /></a>
<a href="http://git.reviewboard.kde.org/r/100920/s/107/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/03/22/new_password_dialog_box_400x100.png" style="border: 1px black solid;" alt="New Password Dialog" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>