<table><tr><td style="">mglb created this revision.<br />mglb added a reviewer: Konsole.<br />mglb added a project: Konsole.<br />mglb requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12058">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><tt style="background: #ebebeb; font-size: 13px;">_preserveLineBreaks</tt> was set/reset only in mousePressEvent, and was<br />
initially disabled. This led to use of the last setting from mouse<br />
selection, or initial value, by "Select All".</p>

<p>BUG 352616</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Start Konsole</li>
<li class="remarkup-list-item">Run <tt style="background: #ebebeb; font-size: 13px;">seq 100</tt></li>
<li class="remarkup-list-item"><em>(optional, test should give the same effect with and without this step)</em> Press <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Ctrl</kbd> and select some text</li>
<li class="remarkup-list-item">Edit → Select All</li>
<li class="remarkup-list-item">Copy</li>
<li class="remarkup-list-item">Paste in any text editor</li>
</ul>

<p><strong>Expected result:</strong> text should be in multiple lines (like on Konsole screen)<br />
<strong>Actual result:</strong> text is pasted as one line</p>

<h4 class="remarkup-header">Regression tests</h4>

<ul class="remarkup-list">
<li class="remarkup-list-item">Run <tt style="background: #ebebeb; font-size: 13px;">seq 100</tt></li>
<li class="remarkup-list-item">Select few lines using mouse</li>
<li class="remarkup-list-item">Copy</li>
<li class="remarkup-list-item">Paste in any text editor</li>
</ul>

<p><strong>Expected result:</strong> text should be in multiple lines (like on Konsole screen)</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Run <tt style="background: #ebebeb; font-size: 13px;">seq 100</tt></li>
<li class="remarkup-list-item">Press <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Ctrl</kbd> and select few lines using mouse</li>
<li class="remarkup-list-item">Copy</li>
<li class="remarkup-list-item">Paste in any text editor</li>
</ul>

<p><strong>Expected result:</strong> text should be pasted as one line</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>arc/Always-preserve-line-breaks-on-Select-All-action (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12058">https://phabricator.kde.org/D12058</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/SessionController.cpp<br />
src/TerminalDisplay.cpp<br />
src/TerminalDisplay.h</div></div></div><br /><div><strong>To: </strong>mglb, Konsole<br /><strong>Cc: </strong>Konsole, herrold, ngraham, hindenburg<br /></div>