<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 22nd, 2016, 4:47 p.m. UTC, <b>Boudhayan Gupta</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;">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>
</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;">Ah sorry, this patch is indeed meant for spectacle (as successor of ksnapshot). I'll move it there in case the general idea is accepted.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">About usability: for people who would like to explore their system, I think it is far easier to select from a ready-made list instead of using the package manager. At least for YaST which I use here, I see no straight-forward way of getting a list of alternative apps to the default one if I don't know exactly for what I am looking for.</p></pre>
<br />
<p>- Gregor</p>
<br />
<p>On January 22nd, 2016, 9:41 a.m. UTC, 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, 9:41 a.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>