<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/103861/">http://git.reviewboard.kde.org/r/103861/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 8th, 2012, 10:20 a.m., <b>Jekyll Wu</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;">Well, after re-reading and re-considering this wish for some time, I feel that original report is a mixture of two related but different issues:
1). Users are confused by the difference between Selection and Clipboard in X11.
2). Users want the handy feature of "copy selected text into clipboard automatically when using Konsole"
For issue 1), I would say: If a user is confused or annoyed by X11 Selection, the best and easiest solution is just ignore and never use that feature. Do copying and pasting in the same explicit way as in other systems. No unique feature, and no confusion.
For issue 2), It is tricky.
First, it is generally problematic by nature. As the reporter and Adam have mentioned, that means "I can't replace the selected text using previously copied text". That would be a big problem for a text editor. However, its impact for Konsole is much smaller or almost zero because you generally do not perform that kind of operation in Konsole.
Second, I notice that Adam mostly ignores Selection except when using Konsole. That is interesting. I guess one main reason is when using Konsole you can't copy selected text into clipboard quickly and explicitly using the well known "Ctrl+C" shortcut. You either use mouse to do it from the context menu(which is slow compared to keyboard), or use some shortcut other than "Ctrl+C" (which is also slow because your muscle remembers Ctrl+C much much better).
So my understanding is that "Ctrl+C" problem is the start point of Adam's suggestion and patch. If so, the patch only needs to introduce one check box : "copy selected text into....". Then you can use "Shift+Insert" consistently for pasting in Konsole.
The extra option for changing the semantic of middle-click-pasting is cool and might be handy(I personally like it), but it is too big a step for only Konsole to provide that unconventional option itself.
</pre>
</blockquote>
<p>On March 18th, 2012, 3:34 p.m., <b>Kurt Hindenburg</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;">Adam, do you agree w/ Jekyll's analyse? I would tend to agree that having the one checkbox is the way to go for now.</pre>
</blockquote>
<p>On March 18th, 2012, 10:23 p.m., <b>Adam Batkin</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;">I would agree that a checkbox to copy to the clipboard on selection (in addition to X Selection) would be fantastic, and would make me very happy (let's say 95%).
But back to #1, in my case, it's not just a case of not using that feature. I'd love to do that, but that's not possible. If I explicitly copy some text ("A") to the clipboard in my word processor, but then I highlight some other text ("B") to make it bold, the word processor has overwritten the X Selection, whether I like it or not. Clipboard=A, Selection=B. So now if I middle click in konsole, it pastes B even though I really wanted A.
I'm not confused by the X Selection, I'm annoyed by it. I use too many operating systems on a daily basis to seamlessly make any use of it, since I'm used to being able to highlight text whenever I want, without the fear of it causing any side-effects.
So to make me 100% happy (well, I'd also like a pot of gold, and a few other things), a second checkbox/radio box would be nice, allowing the user to decide where middle-click pastes from: Clipboard or X Selection (the default, since that would follow the Principle of Least Surprise).</pre>
</blockquote>
<p>On March 31st, 2012, 12:51 p.m., <b>Kurt Hindenburg</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;">Adam, do you have time to just add the checkbox in the mouse section of the Advanced tab? I think there is enough space for it there.</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;">Well, maybe it is a good idea to move "Mouse interaction" into its own tab page, no matter whether we have enouth space in the "Advanced" tab. But if making that change, it is better to eliminate the horizontal scrolling symbol.
</pre>
<br />
<p>- Jekyll</p>
<br />
<p>On March 6th, 2012, 12:48 a.m., Adam Batkin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Konsole.</div>
<div>By Adam Batkin.</div>
<p style="color: grey;"><i>Updated March 6, 2012, 12:48 a.m.</i></p>
<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;">* Creates new "Mouse" Profile Options tab with all of the mouse stuff
* Selected text can now go to X Selection AND Clipboard (instead of ONLY X Selection)
* Middle-Mouse clicks can paste from X Selection or Profile
The whole rationale here is that I almost never care about the X Selection (and
I'm probably not alone). And in the few cases where I DO care, well, that's my
problem. So this patch leaves all of the defaults as-is (no one will notice a
change) but for people for whom it makes a difference, now they can control
what is going on.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=183490">183490</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/EditProfileDialog.h <span style="color: grey">(c669b7f)</span></li>
<li>src/EditProfileDialog.cpp <span style="color: grey">(cbb4875)</span></li>
<li>src/EditProfileDialog.ui <span style="color: grey">(9208009)</span></li>
<li>src/Profile.h <span style="color: grey">(698019b)</span></li>
<li>src/Profile.cpp <span style="color: grey">(5e0a0e2)</span></li>
<li>src/SessionController.h <span style="color: grey">(46714c0)</span></li>
<li>src/SessionController.cpp <span style="color: grey">(e937753)</span></li>
<li>src/TerminalDisplay.cpp <span style="color: grey">(c26d5a9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103861/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/103861/s/436/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/02/14/konsole-selection-configuration_400x100.png" style="border: 1px black solid;" alt="konsole "Mouse" profile configuration tab" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>