Skip to content

Conversation

BDisp
Copy link
Collaborator

@BDisp BDisp commented Feb 13, 2023

Fixes #2342 - Include a feature to set the margin frame and title colors base on the setting of the Border.BorderBrush and the Border.Background properties. Also added xml documents to the true color.

Pull Request checklist:

  • I've named my PR in the form of "Fixes #issue. Terse description."
  • My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
  • My code follows the Terminal.Gui library design guidelines
  • I ran dotnet test before commit
  • I have made corresponding changes to the API documentation (using /// style comments)
  • My changes generate no new warnings
  • I have checked my code and corrected any poor grammar or misspellings
  • I conducted basic QA to assure all features are working

Copy link
Collaborator

@tig tig left a comment

Choose a reason for hiding this comment

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

Good job.

@BDisp
Copy link
Collaborator Author

BDisp commented Feb 21, 2023

@tig I know you refactored very well the Border class in v2, but I think we can leverage this PR in the v1 and only accept the v2 version on merge conflicts. After doing once the merge conflicts doesn't appear anymore for the older changes. What do you think?

@tig
Copy link
Collaborator

tig commented Feb 24, 2023

@BDisp can you please figure out the unit test failures?

@BDisp
Copy link
Collaborator Author

BDisp commented Feb 24, 2023

@BDisp can you please figure out the unit test failures?

@tig it was fixed. There was some merge that cause this not properly work but now is ok. Thanks.

@tig tig merged commit ed27696 into gui-cs:develop Feb 25, 2023
@BDisp BDisp deleted the border-borderbrush-background-fix_2342 branch February 25, 2023 22:53
tig added a commit to tig/Terminal.Gui that referenced this pull request Feb 26, 2023
BDisp pushed a commit to BDisp/Terminal.Gui that referenced this pull request Feb 27, 2023
BDisp added a commit to BDisp/Terminal.Gui that referenced this pull request Mar 1, 2023
…derBrush. (gui-cs#2345)

* Fixes 2342. Border: can't change border color in window by Border.BorderBrush.

* Fixes border initialization.

---------

Co-authored-by: Tig <tig@users.noreply.github.com>
tznind pushed a commit to tznind/gui.cs that referenced this pull request May 7, 2023
…derBrush. (gui-cs#2345)

* Fixes 2342. Border: can't change border color in window by Border.BorderBrush.

* Fixes border initialization.

---------

Co-authored-by: Tig <tig@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Border: can't change border color in window by Border.BorderBrush

2 participants