Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

confirm: properly clear the terminal in case of a multi-line prompt string (take 2) #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

confirm: properly clear the terminal in case of a multi-line prompt string (take 2) #175

wants to merge 3 commits into from
+11 −5

Commits on May 6, 2022

  1. theme.clear(): also clear the current line. 

    This was a simple oversight in the code.
Make sure to preserve the cursor at the beginning of the current line.
    @grunweg
    grunweg committed May 6, 2022
    Configuration menu
    Copy the full SHA
    c4fdc9c View commit details
    Browse the repository at this point in the history

  2. confirm: properly clear the terminal in case of a multi-line prompt s… 

    …tring.
    @grunweg
    grunweg committed May 6, 2022
    Configuration menu
    Copy the full SHA
    c0575d1 View commit details
    Browse the repository at this point in the history

  3. Remove clear_line() calls preceding TermThemeRenderer.clear() in the … 

    …input prompt.

With TermThemeRenderer.clear() also clearing the current line, they have
become superfluous.
    @grunweg
    grunweg committed May 6, 2022
    Configuration menu
    Copy the full SHA
    1be630a View commit details
    Browse the repository at this point in the history