<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/123873/">https://git.reviewboard.kde.org/r/123873/</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;">I'd rather you dropped this. KSnapshot is essentially dead, and I'm not sure we want to do something like this in Spectacle either. Looking for alternative apps is something you'd do in the app store, not inside the application itself. Posting alternative apps in the application will only serve to confuse users.</p></pre>
<br />
<p>- Boudhayan Gupta</p>
<br />
<p>On January 22nd, 2016, 3:11 p.m. IST, Gregor Mi 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 KDE Graphics, KSnapshot, andreas kainz, Boudhayan Gupta, and Matthias Klumpp.</div>
<div>By Gregor Mi.</div>
<p style="color: grey;"><i>Updated Jan. 22, 2016, 3:11 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ksnapshot
</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;">Adds an "Alternatives" menu to the Help menu which contains other screenshot-taking applications.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So, instead of searching the web for alternative applications, the user can find good, hand-picked alternatives directly from within ksnapshot. Applications which are not installed, will be placed in the "More" submenu.</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>CMakeLists.Sources.txt <span style="color: grey">(522fc3f0f943e5c2856c76e93174d39661ca9c50)</span></li>
<li>CMakeLists.txt <span style="color: grey">(cdab928a7db0e7ea29fba35880250639461dbb1e)</span></li>
<li>ksnapshot.h <span style="color: grey">(71ecd6f98485ee8dc488fa8ad8370dc8ca6786b5)</span></li>
<li>ksnapshot.cpp <span style="color: grey">(ac0f2c4f44d47e1a4ae4a318382253a23ad1ed4a)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/123873/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/05/21/7246e0b6-8f31-4183-9f8b-3f52643c040d__alternatives_menu.png">alternatives menu.png</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>