<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/D11893">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The lines are resized when something is put in them. Resizing them<br />
here filled them with spaces (default value), later those trailing<br />
spaces were put into history, and eventually into saved output or<br />
clipboard (because it is assumed lines in history do not contain<br />
trailing spaces).</p>

<p>BUG: 392419</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Enable "Trim trailing spaces" in Profile Settings → Mouse</li>
<li class="remarkup-list-item">Set window size to about 100x100px</li>
<li class="remarkup-list-item">Clear terminal (Ctrl-Shift-K by default)</li>
<li class="remarkup-list-item">Run:</li>
</ul>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">seq $(( $(tput lines) * 2 )); \
printf '*** resize ***'; read -s; printf '\n'; \
seq $(( $(tput lines) * 2 ))</pre></div>

<ul class="remarkup-list">
<li class="remarkup-list-item">When "resize" show up, resize the window to about 500x500px</li>
<li class="remarkup-list-item">Press enter</li>
<li class="remarkup-list-item">Select and copy lines below <tt style="background: #ebebeb; font-size: 13px;">*** resize ***</tt></li>
<li class="remarkup-list-item">Paste it in a text editor which allows to show trailing spaces</li>
</ul>

<p>Expected result: lines should not contain trailing spaces<br />
Actual result: lines added when resizing the window have trailing spaces</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>wip/do-not-resize-new-screen-lines (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11893">https://phabricator.kde.org/D11893</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/Screen.cpp</div></div></div><br /><div><strong>To: </strong>mglb, Konsole<br /><strong>Cc: </strong>Konsole, herrold, ngraham, hindenburg<br /></div>