<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/122753/">https://git.reviewboard.kde.org/r/122753/</a>
</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 Moritz Molch.</div>
<p style="color: grey;"><i>Updated March 2, 2015, 10:35 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;">Thank you for your patience! I think I fixed all the issues and I hope that the patch is OK now. Speaking of border-cases, another one could be floating docks. The people at kdenlive never hide the titlebars in that case but at least for me, hiding them even in that case has proven to work very well for me while drawing in krita.</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;">Inspired by kdenlive, this small patch adds the option to show/hide the titlebars of all dockers to the settings menu.</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 on Ubuntu 14.04</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>krita/ui/kis_config.cc <span style="color: grey">(a09c7b8)</span></li>
<li>libs/main/KoMainWindow.h <span style="color: grey">(ac6cc82)</span></li>
<li>libs/main/KoMainWindow.cpp <span style="color: grey">(b66fb49)</span></li>
<li>libs/main/calligra_shell.rc <span style="color: grey">(3f09d7e)</span></li>
<li>libs/widgets/KoDockWidgetTitleBar.cpp <span style="color: grey">(cf0c722)</span></li>
<li>krita/krita.action <span style="color: grey">(b683185)</span></li>
<li>krita/krita.rc <span style="color: grey">(0ec0931)</span></li>
<li>krita/ui/KisMainWindow.h <span style="color: grey">(9cdcce5)</span></li>
<li>krita/ui/KisMainWindow.cpp <span style="color: grey">(e8c6009)</span></li>
<li>krita/ui/kis_config.h <span style="color: grey">(b6852de)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/122753/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>