<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/123486/">https://git.reviewboard.kde.org/r/123486/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="12" style="border: 1px gray solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<h1 style="margin: 0; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By Dmitry Kazakov.</div>
<p style="color: grey;"><i>Updated April 25, 2015, 10:16 a.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;">Submitted with commit 0616ea1f9bf958bcce5703a1bd65f4bfe11edce3 by Dmitry Kazakov to branch calligra/2.9.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch fixes numerous bugs in KoResourceItemChooser</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">1) Resizing the widged and then hide/show used to reset currently selected resource. That caused user's paintop presets and styles being invalidated by simply showing the widget on screen. Now this problem has gone, because the madoel()->reset() call is done in two stages with first saving the current selection and then restoring it when the reset is finished.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">2) Now the number of columns is calculated dynamically when the widget is being resized. Before the patch there was a weird behaviour: when resizing the widget the tiles were <strong style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">scaled only</strong>, and after the next hide/show the <strong style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">number of columns</strong> would be recalculated. Now both steps are executed simultaneously.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">3) The size of the preview splitter in KisPatternChooser is now fixed, so the user would see at least something :)</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tested in Krita only</p></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>libs/widgets/KoResourceModel.cpp <span style="color: grey">(3b2f5e3)</span></li>
<li>libs/widgets/KoResourceModel.h <span style="color: grey">(06a7135)</span></li>
<li>libs/widgets/KoResourceItemView.cpp <span style="color: grey">(e5d9e5a)</span></li>
<li>libs/widgets/KoResourceItemView.h <span style="color: grey">(9b712ed)</span></li>
<li>libs/widgets/KoResourceItemChooser.cpp <span style="color: grey">(6e5c349)</span></li>
<li>libs/widgets/KoResourceItemChooser.h <span style="color: grey">(e41bd26)</span></li>
<li>krita/ui/widgets/kis_pattern_chooser.cc <span style="color: grey">(9568d3b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/123486/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>