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






<blockquote style="margin: 1em 0 0 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On December 7th, 2012, 12:08 p.m., <b>Bart Cerneels</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding: 0 0 0 1em;">
  
  <br style="display: none;" />

  <table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/881/" style="color: black; font-weight: bold; font-size: 9pt;">new-guess-tags.png</a></td>
   </tr>
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/881/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-guess-tags_37_95_236_115.png" style="border: 1px black solid;" alt="New Guess Tag Dialog"></a></td>
   </tr>
  </table>
  <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 colors here cause a readability issue (yellow on white in this case). Since the background color of the view depends on the style and color settings it's impossible to find colors that will work in all cases. Just use the color settings forground-text, no special colors.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, the old version has the same problems.
Also the preview will show wrong colors in some border conditions.
I left it because I didn't have any better ideas</pre>
<br />

<blockquote style="margin: 1em 0 0 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On December 7th, 2012, 12:08 p.m., <b>Bart Cerneels</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding: 0 0 0 1em;">
  
  <br style="display: none;" />

  <table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/883/" style="color: black; font-weight: bold; font-size: 9pt;">new-edit-filter.png</a></td>
   </tr>
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/883/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-edit-filter_26_303_390_16.png" style="border: 1px black solid;" alt="New Edit Filter Dialog"></a></td>
   </tr>
  </table>
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">What does this line edit do? Needs a label.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">That is the filter result. Editable. Perviously we had there the big box.</pre>
<br />

<blockquote style="margin: 1em 0 0 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On December 7th, 2012, 12:08 p.m., <b>Bart Cerneels</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding: 0 0 0 1em;">
  
  <br style="display: none;" />

  <table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/883/" style="color: black; font-weight: bold; font-size: 9pt;">new-edit-filter.png</a></td>
   </tr>
   <tr>
    <td><a href="http://git.reviewboard.kde.org/r/107624/s/883/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-edit-filter_26_268_513_17.png" style="border: 1px black solid;" alt="New Edit Filter Dialog"></a></td>
   </tr>
  </table>
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If the option count < 4, don't use a combo. And don't make it this wide in any case.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I didn't make it a combo, but you are right. Radio boxes would probably be better.</pre>
<br />






<p>- Ralf</p>


<br />
<p>On December 7th, 2012, 11:44 a.m., Ralf Engels wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Ralf Engels.</div>


<p style="color: grey;"><i>Updated Dec. 7, 2012, 11:44 a.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;">Several refactoring changes centering around token pool, token, token drop target and so on.

1. splitting up the FilenameLayoutDialog (which was not a dialog at all) into two widgets to be used by the actual dialog.
2. changing the token pool from an icon list to a normal list to prevent the use from having to scroll so much.
3. simplifying the drop target so that it does not need to install event filters for it's parents.
4. fixing small issues in the token pool, token and token drop target so that they have sensible minimumSizeHint and sizeHints
5. aligning texts between the different token users. No longer different texts.
6. for the edit filter dialog changes to the layout to get rid of the space waste in the result area.
7. for the guess tag dialog I got rid of some empty areas and some useless settings (settings that the dialog could determine itself)
8. for the playlist layout dialog not much has changed except that we don't need the event-filter parent mechanism.

Have a look at the attached screenshots to see the differences.
The actual code changes are in the rengels-filenameLayoutDialog branch.

Further work: Settings and presets need a reworking, as we have several sets currently which is confusing.
</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">(15a698c)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/107624/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/107624/s/878/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/old-layout-editor_400x100.png" style="border: 1px black solid;" alt="Old Layout editor dialog" /></a>

 <a href="http://git.reviewboard.kde.org/r/107624/s/879/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-layout-editor_400x100.png" style="border: 1px black solid;" alt="New Layout editor dialog" /></a>

 <a href="http://git.reviewboard.kde.org/r/107624/s/880/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/old-guess-tags_400x100.png" style="border: 1px black solid;" alt="Old Guess Tag Dialog" /></a>

 <a href="http://git.reviewboard.kde.org/r/107624/s/881/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-guess-tags_400x100.png" style="border: 1px black solid;" alt="New Guess Tag Dialog" /></a>

 <a href="http://git.reviewboard.kde.org/r/107624/s/882/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/old-edit-filter_400x100.png" style="border: 1px black solid;" alt="Old Edit Filter Dialog" /></a>

 <a href="http://git.reviewboard.kde.org/r/107624/s/883/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/07/new-edit-filter_400x100.png" style="border: 1px black solid;" alt="New Edit Filter Dialog" /></a>

</div>


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








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