Modify the display in multi-line#350
Closed
godlockin wants to merge 4 commits into
Closed
Conversation
…ent per line - Change default flexMode from 'full-minus-40' to 'full' so status line content is not unnecessarily truncated for new/default configurations - Refactor calculateMaxWidthsFromPreRendered to return number[][] instead of number[], giving each line its own independent column widths so that a wide widget on one line no longer forces the same column wider on other lines when autoAlign is enabled Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
…default-flex-mode Fix/independent line widths and default flex mode
sirmalloc
force-pushed
the
main
branch
3 times, most recently
from
4f7a07b to
ec28376
Compare
Owner
|
@godlockin Can you explain the intent of this PR a little more clearly? The whole purpose of auto-alignment in powerline mode is to ensure every widget is vertically aligned with the widget above and/or below it. By calculating max width's on a per-line basis, this effectively prevents that functionality from working as intended. A better fix to exclude wide widgets from affecting everything else would be to allow a single widget to be manually flagged as excluded from the auto-align calculation, which would basically cause that widget and everything after it on the same line to be exempt from any changes to their default widths. |
Owner
|
Closing this as a dupe of #349, I'll copy my previous question there. |
Repository owner
locked as resolved and limited conversation to collaborators
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reported behavior: on multi-line status bars, a widget with long content (e.g. a long git branch name) would push adjacent columns wider on other lines. Also, out-of-the-box configurations were truncating content on wider terminals due to an overly conservative default.