Skip to content

fix: support for high contrast #300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: support for high contrast #300

wants to merge 1 commit into from

Conversation

mcoker
Copy link

@mcoker mcoker commented Oct 15, 2025 •
edited
Loading

What

Closes #298

Preview link: https://react-topology-pr-topology-300.surge.sh/

To enable high contrast, you have to do it manually by adding the class pf-v6-theme-high-contrast to the document's <html> element.

Description

  • Updates default group stroke color to border--color--main--default
  • Updates default group stroke width to border--width--extra-strong (3px)
    • I didn't see this called out explicitly (unless I missed it), but seems implied by the notes in figma to make the borders (3px default, x-strong + 1 hover, x-strong + 2 selected)
  • Updates group hover stroke width to 4px in HC only
  • Updates group selected stroke width to 5px in HC only
  • Updates group hover + selected stroke width to 4px in HC only
  • Updates edge stroke from #707070 to border--color--on-secondary

Also updates the PF packages to latest prerelease, I imagine we'll remove that and bump to 6.4.0 here or in a separate PR once the PF release lands.

Type of change

  • Feature
  • Bugfix
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Screen shots / Gifs for design review

@patternfly-build
Copy link

Preview: https://react-topology-pr-topology-300.surge.sh

@mcoker
Copy link
Author

mcoker commented Oct 15, 2025

@lboehling - tagging you here since I can't add you as a reviewer.

@mcoker mcoker mentioned this pull request Oct 15, 2025
Open
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrew-ronaldson andrew-ronaldson Awaiting requested review from andrew-ronaldson

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

High contrast: Update tokens to enable high contrast theme

2 participants

@mcoker @patternfly-build