Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Improve the size of completions page to show the entire multi-line prompt#4702

Closed
s417-lama wants to merge 1 commit into
fish-shell:masterfish-shell/fish-shell:masterfrom
s417-lama:improve_completions_page_sizes417-lama/fish-shell:improve_completions_page_sizeCopy head branch name to clipboard
Closed

Improve the size of completions page to show the entire multi-line prompt#4702
s417-lama wants to merge 1 commit into
fish-shell:masterfish-shell/fish-shell:masterfrom
s417-lama:improve_completions_page_sizes417-lama/fish-shell:improve_completions_page_sizeCopy head branch name to clipboard

Conversation

@s417-lama

Copy link
Copy Markdown
Contributor

Description

Problem

When editing multi lines and selecting completions, the page of completions hides the lines except the bottom line.
I think it is uncomfortable not to be able to see the entire lines even after selecting the completion.

What I changed

I changed to show the entire lines when selecting completions.

TODOs:

  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.md

@ridiculousfish

Copy link
Copy Markdown
Member

Thanks, nice fix! I'm going to merge it as is, but then I will change it to use a simple newline count instead of the heavyweight prompt layout machinery.

Merged as 27c1c06 - thanks!

@ridiculousfish ridiculousfish added this to the fish-3.0 milestone Feb 4, 2018
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.