<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/120346/">https://git.reviewboard.kde.org/r/120346/</a>
</td>
</tr>
</table>
<br />
<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;">Yes, I think this is a good thing to do. But creating the tooltip in this way only works for the default shortcuts. If you reassign, say, the shortcut for the freehand tool to 'F', the tooltip will still say 'B' even after a restart. I think Sven had some ideas on making this more dynamic.</p></pre>
<br />
<p>- Boudewijn Rempt</p>
<br />
<p>On September 24th, 2014, 3 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.</div>
<div>By Scott Petrovic.</div>
<p style="color: grey;"><i>Updated Sept. 24, 2014, 3 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;">I think this will make the shortcuts more discoverable</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/tools/defaulttools/kis_tool_brush.h <span style="color: grey">(885870c)</span></li>
<li>krita/plugins/tools/selectiontools/kis_tool_select_rectangular.h <span style="color: grey">(fdd70b8)</span></li>
<li>krita/plugins/tools/tool_transform2/kis_tool_transform.h <span style="color: grey">(aa1660d)</span></li>
<li>krita/plugins/tools/defaulttools/kis_tool_gradient.h <span style="color: grey">(09c60c7)</span></li>
<li>krita/plugins/tools/defaulttools/kis_tool_move.h <span style="color: grey">(79b2058)</span></li>
<li>krita/plugins/tools/defaulttools/kis_tool_multihand.h <span style="color: grey">(512f255)</span></li>
<li>krita/plugins/tools/selectiontools/kis_tool_select_elliptical.h <span style="color: grey">(0ce8610)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120346/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>