Skip to content

Conversation

@wugeer
Copy link
Contributor

@wugeer wugeer commented Aug 30, 2024

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
Copy link
Collaborator

@cruessler cruessler left a comment

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
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 <[email protected]>
@wugeer
Copy link
Contributor Author

wugeer commented Nov 3, 2025

@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