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.

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

Extra line on Ctrl + C on 2 line prompt (low priority) #53

Closed
TheToto opened this issue Dec 12, 2020 · 3 comments
Closed

Extra line on Ctrl + C on 2 line prompt (low priority) #53

TheToto opened this issue Dec 12, 2020 · 3 comments
Labels
🌊 enhancement New feature or request

Comments

@TheToto
Copy link

TheToto commented Dec 12, 2020

Describe the bug

When I press Ctrl + C on a empty prompt, it simply redraw the prompt.
When I press Ctrl + C on a prompt with some text, it redraw the prompt with a extra unwanted line (see screenshots)

Steps to reproduce

  1. Use a 2 line prompt
  2. Type something
  3. Press Ctrl + C

Expected behavior

A clean redraw

Screenshots

Before Ctrl + C
image
After Ctrl + C
image

Environment (please complete the following information)

  • Operating System: ArchLinux
  • Output of tide bug-report:
fish, version 3.1.2
tide, version 2.5.0
xterm-256color
@TheToto TheToto added the 🐛 bug Something isn't working label Dec 12, 2020
@IlanCosman
Copy link
Owner

IlanCosman commented Dec 12, 2020

This is an upstream bug. The same issue will appear with any multi-line prompt:

function fish_prompt
    echo foo
    echo bar
end

@IlanCosman IlanCosman added the 🏞️ upstream Due to upstream constraints label Dec 12, 2020
@IlanCosman IlanCosman removed the 🏞️ upstream Due to upstream constraints label Dec 18, 2020
@IlanCosman
Copy link
Owner

I'm removing upstream because this isn't really a bug from fish's perspective. I found a way to work around fish's behavior, so this issue will be fixed after the 3.0 release.

@IlanCosman IlanCosman added 🌊 enhancement New feature or request and removed 🐛 bug Something isn't working labels Dec 22, 2020
@IlanCosman
Copy link
Owner

IlanCosman commented Dec 23, 2020

It seems I messed up again here 😂 This was a bug from the fish maintainer's perspective: fish-shell/fish-shell#3537. It's fixed for the 3.2 release.

Even more strangely is that fish 3.1.2 behaves differently on MacOS and Linux. On MacOS, things seem to be working just fine for multi-line prompt cancellation, while on Linux we see the problem this issue described. Thus, the changes I made for this issue fix things on Linux, but then make MacOS worse 🤦‍♂️

So I've reverted the changes meant to fix this problem. Rules for next time: check upstream first, and test both OSes.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🌊 enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants