<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/128726/">https://git.reviewboard.kde.org/r/128726/</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;">As the maintainer of both klipper and kglobalaccel I'm strictly against ading this shortcut by default! This was already stated by me in the bug report by setting it to wontfix.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">At the moment this would result in a hidden default shortcut no user knows about. This is bad, we should not add unexepected and unknown shortcuts. And we should not needlessly steal shortcuts from applications. In doubt don't set a shortcut by default. At the current time this would only result in a degrated user experience: either an in-application shortcut, which we don't know that it existed, started to break, or the user presses the shortcut and gets hit by surprise what happens. It's totally unexpected behavior to open this menu.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Before we start to add new shortcuts, we need to rethink and implement better ways of global shortcut handling:
1. educating users about shortcuts
2. ensuring that we don't steal shortcuts (e.g. by setting up a policy about which modifier belongs to the DE)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">As long as these problems are not solved, I'm strictly against adding shortcuts for hidden functionality. No matter how much we personally think they are sensible.</p></pre>
<br />
<p>- Martin Gräßlin</p>
<br />
<p>On August 21st, 2016, 2:47 a.m. CEST, Andreas Kainz 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 Plasma and KDE Usability.</div>
<div>By Andreas Kainz.</div>
<p style="color: grey;"><i>Updated Aug. 21, 2016, 2:47 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-workspace
</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;">add a default shortcut for open clipboard at mouse point (ctrl+alt+c)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">BUG 366690</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>klipper/klipper.cpp <span style="color: grey">(525ef87)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128726/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>