<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/110276/">http://git.reviewboard.kde.org/r/110276/</a>
     </td>
    </tr>
   </table>
   <br />




<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, KDE Usability, David Edmundson, and Martin Klapetek.</div>
<div>By Sven Brauch.</div>


<p style="color: grey;"><i>Updated May 3, 2013, 1:21 p.m.</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">I added the access keys (I would never have noticed, thanks), and reordered the boxes in a way I think makes sense.

> At least the "now playing" edit field has a height without top/bottom space.
Sorry, I'm still not sure what you mean here. :(
It does have the same height like any other line edit, doesn't it?
I added a label to make it clear what the field even does (see new screenshot). I think the preset is reasonable enough so that people who consider it too complicated just don't have to touch it at all.

> My conclusion would be to prevent closing a window without any feature to reactivate. Or to confirm per dialog in case there to close means to shut down.
Yeah, I don't think the current behaviour is ideal either. However I think this is a different issue and should be discussed seperately. ;)

> What's about the idea to separate the "Now playing" stuff, with the benefit of room for support?
To be honest, I don't like it. It splits the dialog in two pages, which tends make it more difficult to find the option you're looking for quickly (given that the options would also fit on one page of course -- which they do).
Besides that, I agree that the "Now playing" thing looks a bit difficult at first glance, but if you look at it for a moment, from the preset "Now listening to %title by %artist from album %album" it should be pretty obvious how to modify the message the way you want it to look like. And if it's not obvious to someone, he doesn't have to touch it. To put it differently, I wouldn't even know what kind of instructions to put there... "%artist will be replaced by the artist of the currently played album"? Do you think that would help anyone (honest question, because I don't think it would)?

> More than 700x640?!
Heh, yes -- the first screenshot I posted shows the previous and new default size of the dialog.
I should maybe mention that this will only set the default size the dialog has when it first opens, so the user can still resize it if he wants.

> Isn't it possible to keep the size that users have adjusted?
Possible yes, but it needs extra code and I don't think it's worth it. This is the kind of settings dialog you use maybe once or twice per installation... when you start using the software, you use it to set the options you want once, and then you never look at it again. At least that's what I imagine (or how I would use it if I weren't working on its layout).

Thanks for your thoughts!
Cheers,
Sven</pre>
  </td>
 </tr>
</table>




<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;">This patch revamps the settings UI of the kded module. Especially, it removes the "flat" flag from the group boxes, and also removes lots of spacers. But talk doesn't really help with UI changes, so you'd better look at the screenshot below ;)

Additionally, I'd like to add a dialog->resize(700, 640) to ContactListWidget::showSettingsKCM() in contact-list-widget.cpp in ktp-contactlist (the function which opens the dialog), but I don't want to open a seperate review for that.</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;">The settings dialog still seems to work fine.</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>config/telepathy-kded-config.cpp <span style="color: grey">(50c176d)</span></li>

 <li>config/telepathy-kded-config.ui <span style="color: grey">(54ebc54)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments (updated)</h1>

<ul>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/02/ktp-settings.png">Old (left) and new (right) settings dialog, with their respective default sizes</a></li>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/03/ktp-settings-new2.png">new settings dialog as of diff revision 2</a></li>

</ul>





  </td>
 </tr>
</table>




  </div>
 </body>
</html>