<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/108991/">http://git.reviewboard.kde.org/r/108991/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit f0903c2bf3bdcd3db3198e02ad8148eff7db6e73 by David Edmundson on behalf of Stefan Eggers to branch master.</pre>

 <br />

<p>- Commit</p>

<br />

<p>On February 17th, 2013, 12:44 p.m. UTC, Stefan Eggers wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for Telepathy.</div>

<div>By Stefan Eggers.</div>

<p style="color: grey;"><i>Updated Feb. 17, 2013, 12:44 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;">set focus on the password field

Previously when the dialog opened the focus was on the OK button and

one had to click on the KLineEdit widget to enter the password. With

this change the foucs is on the KLineEdit widget instead and one can

start to enter the password right away.

In addition to this make sure the dialog doesn't steal focus. Other-

wise it might interrupt text input in some other window when it opens.</pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Testing without this change showed that the OK button had focus when the password dialog opened. With this change the input field for the password has focus.

As for the issue of the dialog stealing the focus I tested logging in with a KDE session which previously had Konsole open. I made sure the password dialogs were not open when logging out. After Konsole starts I begin typing some random text on the command line while waiting for the password dialog to open. The password dialog opened in the background and did not interrupt text input in Konsole with this change.</pre>

  </td>

 </tr>

</table>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>

 <a href="http://bugs.kde.org/show_bug.cgi?id=312338">312338</a>

</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>

<ul style="margin-left: 3em; padding-left: 0;">

 <li>x-telepathy-password-prompt.cpp <span style="color: grey">(7f2be8117e0d0a98c2f345780e7bc19831697a89)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/108991/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>