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

[Tui] Fix invisible border with null color in BorderPattern's inverse strategies#64223

Merged
nicolas-grekas merged 1 commit into
symfony:8.1symfony/symfony:8.1from
sblondeau:fix/border-patternsblondeau/symfony:fix/border-patternCopy head branch name to clipboard
May 20, 2026
Merged

[Tui] Fix invisible border with null color in BorderPattern's inverse strategies#64223
nicolas-grekas merged 1 commit into
symfony:8.1symfony/symfony:8.1from
sblondeau:fix/border-patternsblondeau/symfony:fix/border-patternCopy head branch name to clipboard

Conversation

@sblondeau
Copy link
Copy Markdown
Contributor

Q A
Branch? 8.1
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

fix BorderPattern issue for WIDE, WIDE_MEDIUM, TALL, TALL_MEDIUM
Before fix, white border appears :
image

After fix:
image

@carsonbot carsonbot added this to the 8.1 milestone May 15, 2026
@carsonbot carsonbot changed the title [TUI] Fix description of the bug in BorderPattern [Tui] Fix description of the bug in BorderPattern May 15, 2026
Copy link
Copy Markdown
Member

@GromNaN GromNaN left a comment

Choose a reason for hiding this comment

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

Thanks for the screenshots, that really helps to understand the bug and the fix. Would it be possible to add a test in BorderPatternTest?

@symfony symfony deleted a comment from carsonbot May 20, 2026
@nicolas-grekas nicolas-grekas changed the title [Tui] Fix description of the bug in BorderPattern [Tui] Fix invisible border with null color in BorderPattern's inverse strategies May 20, 2026
@nicolas-grekas
Copy link
Copy Markdown
Member

Thank you @sblondeau.

@nicolas-grekas nicolas-grekas merged commit 7b1134c into symfony:8.1 May 20, 2026
8 of 12 checks passed
@fabpot fabpot mentioned this pull request May 27, 2026
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.

4 participants

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