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

Comments

Close side panel

fix(HeaderToggle): updated border radius in high contrast#730

Merged
rebeccaalpert merged 2 commits intopatternfly:mainpatternfly/chatbot:mainfrom
thatblindgeye:iss698thatblindgeye/chatbot:iss698Copy head branch name to clipboard
Nov 3, 2025
Merged

fix(HeaderToggle): updated border radius in high contrast#730
rebeccaalpert merged 2 commits intopatternfly:mainpatternfly/chatbot:mainfrom
thatblindgeye:iss698thatblindgeye/chatbot:iss698Copy head branch name to clipboard

Conversation

@thatblindgeye
Copy link
Collaborator

Closes #698

@patternfly-build
Copy link

patternfly-build commented Oct 24, 2025

@rebeccaalpert
Copy link
Member

Copy link
Member

@rebeccaalpert rebeccaalpert left a comment

Choose a reason for hiding this comment

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

Works for me!

justify-content: center;

&::before {
border-radius: var(--pf-t--global--border--radius--pill);
Copy link
Member

Choose a reason for hiding this comment

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

I think you could also use inherit as the value here; that's what we tended to use in core so that the pseudoelements match the element border radius without having to use the variable again.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good call out, pushed that update!

Copy link

@lboehling lboehling left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Member

@srambach srambach left a comment

Choose a reason for hiding this comment

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

L 🤖 TM!

@rebeccaalpert rebeccaalpert merged commit 5990ef8 into patternfly:main Nov 3, 2025
7 checks passed
@github-actions
Copy link

github-actions bot commented Nov 3, 2025

🎉 This PR is included in version 6.5.0-prerelease.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Buttons high contrast

5 participants

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