[konsole] [Bug 384218] clear/^l shouldn't add empty lines to the scrollback buffer

[Kurt Hindenburg]

https://bugs.kde.org/show_bug.cgi?id=384218

Kurt Hindenburg <kurt.hindenburg at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/kde/ |https://commits.kde.org/kon
                   |konsole/commit/7a1116ccff5d |sole/7a1116ccff5da88f6ef7e0
                   |a88f6ef7e0c4d50ae979b09fed8 |c4d50ae979b09fed8f
                   |f                           |

--- Comment #10 from Kurt Hindenburg <kurt.hindenburg at gmail.com> ---
Git commit 7a1116ccff5da88f6ef7e0c4d50ae979b09fed8f by Kurt Hindenburg, on
behalf of Mariusz Glebocki.
Committed on 15/06/2019 at 21:03.
Pushed by scmsync into branch 'master'.

CSI 2J: Do not append current screen to history

Summary:
This behavior is present in e.g. xterm and linux console.

Currently when using CSI 2J escape sequence, screen contents are
appended to scrollback (including bottom empty lines). This is breaking
`clear` command, which replaces scrollback with current screen contents.

Test Plan:
* Start Konsole with a shell
* `clear`

**Expected result:**
Cleared display, empty scrollback.
**Actual result:**
Cleared display, but scrollback contains previous display contents.

---

* Start Konsole with a shell
* seq 1000
* `echo -ne '\033[2J`

**Expected result:**
Cleared display, numbers in last scrollback lines.
**Actual result:**
Cleared display, but scrollback contains previous display contents
(i.e. shell prompt in last lines)

Reviewers: #konsole, hindenburg

Reviewed By: #konsole, hindenburg

Subscribers: hindenburg, konsole-devel, #konsole

Tags: #konsole

Differential Revision: https://phabricator.kde.org/D21282

M  +0    -6    src/Screen.cpp

https://commits.kde.org/konsole/7a1116ccff5da88f6ef7e0c4d50ae979b09fed8f

-- 
You are receiving this mail because:
You are the assignee for the bug.