<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/113057/">http://git.reviewboard.kde.org/r/113057/</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 Amarok.</div>
<div>By Ricardo Varas.</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=322016">322016</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
amarok
</div>
<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;">I am trying to kill Bug 322016 - Apply button is always enabled in Playlist Layout Editor dialog.
In multiple parts I call the enable property for the Apply button and enable/disable it according to specific actions. I tried to cover all possible actions from a user --of course it begins in the disabled status as requested and it's disabled after the user clicks it --.
One thing I am not happy about is the fact that when the Token (the option with the tool icon) is used to configure the playlist items such as album, album artist, etc. then the Apply button is always enabled and I believe under that case it should only become enabled if the Configuration for... dialog's OK button is clicked, and remain disabled after the Cancel button is clicked or the dialog is closed. I think LayoutEditDialog.cpp is called every time the token is chosen but right not I am unable to understand how they are connected LayoutEditDialog.cpp and PlaylistLayoutEditDialog.cpp.
I'd appreciate if someone can help me understand this one and put together a good patch for this bug.
Thanks.</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>PlaylistLayoutEditDialog1.cpp <span style="color: grey">(99aee2a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113057/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>