<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/119558/">https://git.reviewboard.kde.org/r/119558/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On August 7th, 2014, 7:29 p.m. UTC, <b>Boudewijn Rempt</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This looks pretty much perfect! Except for one niggle, and I'm not sure about that -- it looks like the actual curve widget is now really small, and the labels sort of disconnect from it. But maybe it always was like that -- check http://www.valdyas.org/~boud/curvewidget.png and tell me whether I'm missing something or not. When I'm sure about that part, ship it!</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There is something funky going on with those curve editors in general. I have a screenshot what it looks like in 2.8.3 on my Windows 7 machine. http://www.scottpetrovic.com/uploads/krita/krita-2.8.3-curve%20editor.jpg It is slightly different than yours, but it is still pretty small. Some settings areas by the curve editor have different padding, so it looks like it changes the curve editor as well. I think the entire "edit brush settings" need to be re-organized a little bit (along with updating the curve editor). Out of all of the areas in the application, I think that could use the most UI improvements. I bought that C++ book you mentioned and am tinkering around with the code in general. I think what I have is ok for now. You're the boss though!</p></pre>
<br />
<p>- Scott</p>
<br />
<p>On August 1st, 2014, 1:54 a.m. UTC, Scott Petrovic wrote:</p>
<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 and Boudewijn Rempt.</div>
<div>By Scott Petrovic.</div>
<p style="color: grey;"><i>Updated Aug. 1, 2014, 1:54 a.m.</i></p>
<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;">add labels for nameless sliders</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>krita/plugins/extensions/dockers/defaultdockers/wdglayerbox.ui <span style="color: grey">(d52b2f8)</span></li>
<li>krita/plugins/paintops/libpaintop/forms/wdgautobrush.ui <span style="color: grey">(2f56bc1)</span></li>
<li>krita/plugins/paintops/libpaintop/forms/wdgcurveoption.ui <span style="color: grey">(ee8d229)</span></li>
<li>krita/plugins/paintops/libpaintop/kis_curve_option_widget.cpp <span style="color: grey">(c3ea0e8)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119558/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>