Skip to content

fix: When the terminal is insufficient to display all the commands, the cmdbar_bg configuration color does not fully take effect#2348

Merged
extrawurst merged 3 commits into
gitui-org:masterfrom
wugeer:feature_xhw
Oct 28, 2025
Merged

Conversation

@wugeer

@wugeer wugeer commented Aug 30, 2024

Copy link
Copy Markdown
Contributor

This Pull Request fixes/closes #2347.

It changes the following:

  • Remove the line field from the Command struct, along with its corresponding initialization
  • Remove the cmdbar_extra_lines_bg field from the Theme struct, along with its corresponding initialization

I followed the checklist:

  • I added unittests
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog

…he cmdbar_bg configuration color does not fully take effect
@extrawurst extrawurst requested a review from cruessler October 22, 2025 09:25

@cruessler cruessler left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I’ve tested it locally and it works regardless of whether I’ve customized cmdbar_bg.

@cruessler

Copy link
Copy Markdown
Collaborator

@wugeer Would you mind resolving the conflict in CHANGELOG.md?

@extrawurst extrawurst added this to the v0.28 milestone Oct 28, 2025
@extrawurst extrawurst merged commit 7674dae into gitui-org:master Oct 28, 2025
22 checks passed
cruessler pushed a commit to cruessler/gitui that referenced this pull request Oct 29, 2025
…he cmdbar_bg configuration color does not fully take effect (gitui-org#2348)

Co-authored-by: extrawurst <776816+extrawurst@users.noreply.github.com>
@wugeer

wugeer commented Nov 3, 2025

Copy link
Copy Markdown
Contributor Author

@wugeer Would you mind resolving the conflict in CHANGELOG.md?

Sorry for the late reply. I just saw this message. I'm glad the issue is resolved without bothering you. Thx! @cruessler @extrawurst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When the terminal is insufficient to display all the commands, the cmdbar_bg configuration color does not fully take effect.

3 participants