<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>
<p>On January 23rd, 2016, 5:20 p.m. UTC, <b>Gregor Mi</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;">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>
</blockquote>
<p>On January 23rd, 2016, 5:28 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;">As the maintainer for both apps, I'm not going to accept this idea.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'll re-iterate why:</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">As I said, it will only serve to confuse users. The place to suggest alternative apps is the package manager or app store, not inside the app itself.</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">If this were accepted (which it won't), all other KDE apps would need to suggest alternatives to themselves for consistency. If not it'd only add to the confusion arising from the previous point, not lessen it.</li>
</ul>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">If you're serious about the idea of suggesting alternative apps, I'd suggest trying to implement something in Plasma, something that suggests alternatives at a more "global" place. The application isn't the place to do this at all.</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;">Hi Boudhayan, thank you for the explanatory statement. Of course, I accept the rejection.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I will propose the idea to another audience (the VDG) and reference this RR as one opinion about this matter.</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>