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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 8th, 2011, 11:43 a.m., <b>Sergey Ivanov</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;">I think that It&#39;s a bad idea to apply any settings when user press Cancel, so storing Format Presets should stay in onAccept slot. 
Agree with second statement.

And finally this patch has nothing in common with mentioned Bug Report.</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;">That&#39;s the thing. Pressing &quot;Save preset&quot; gives the impression that the preset has indeed been saved. It has happened to me a couple of times that I created a new Preset, checked the preview, found something wrong with my tagging, pressed cancel, corrected the tags and then wanted to use the saved preset again which was gone... The way I see it both ways are essentially wrong from a certain Design perspective. 
I am by no means a GUI or Usability expert (my expertise comes down to 3 lectures regarding User Interface Design and Consistency in the user interface and some small Project in University). But what I learned is that you should make Dialogs as unambiguous as possible and with respect to that the discrepancy between the *strong* implication of the preset being &quot;Saved&quot; and then again not when pressing cancel is very hard to resolve.

A solution to that would be to add a third button like &quot;Do not move files but save settings&quot; (Better wording needed :D) or to remove the cancel button altogether and remove it with something like &quot;Close&quot; that does not imply that the settings are discarded and save them anyway... Of the two the first option would probably be preferable as it doesn&#39;t remove the Deesktop Paradigma that Dialogs should have a cancel button that aborts everything it does.

With respect to the bugreport: You&#39;re right, this review request has nothing to do with it, but could you then please comment on the bug why you reopened it? When I was checking in IRC with the person who apparently initiated that I found all these inconsistencies but not that the bug was still there.</pre>
<br />








<p>- Philipp</p>


<br />
<p>On January 8th, 2011, 12:12 p.m., Philipp Schmidt 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 Amarok.</div>
<div>By Philipp Schmidt.</div>


<p style="color: grey;"><i>Updated Jan. 8, 2011, 12:12 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;">Fixes two errors:

First: Presets are being saved explicitely, meaning they should persist even when the Dialog is aborted/canceled.
Second: The state of the Current Collection Directory is saved regardless of whether the Dialog was accepted or canceled. IMO it should only be saved like all other values when it is accepted.</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>ChangeLog <span style="color: grey">(3c337d1)</span></li>

 <li>src/dialogs/OrganizeCollectionDialog.cpp <span style="color: grey">(b7d7850)</span></li>

</ul>

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




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








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