Skip to content

fix(dual-list-selector): tree to support focus-within, hc border on hover/focus - #7829

Merged
mcoker merged 2 commits into
patternfly:mainfrom
mcoker:issue-7795
Sep 15, 2025
Merged

mcoker merged 2 commits into
patternfly:mainfrom
mcoker:issue-7795

Conversation

@mcoker

@mcoker mcoker commented Sep 12, 2025 •

Copy link
Copy Markdown
Contributor

fixes #7795

  • Adds HC border on hover/focus for tree variant
  • Adds background color change when focusing on something (either the expand toggle or checkbox) in a tree item - this is a net new style to the default styles, not just HC - @lboehling is that ok? This matches tree view styles.

Previously looks like this:

Screenshot 2025-09-12 at 2 40 19 PM

Now looks like this:

Screenshot 2025-09-12 at 2 40 56 PM

For comparison this is tree view, which has the same styling:
Screenshot 2025-09-12 at 2 48 31 PM

@patternfly-build

patternfly-build commented Sep 12, 2025 •

Copy link
Copy Markdown
Collaborator

@lboehling lboehling left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm! good catch w the focus states

@srambach srambach left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

L 🤩 TM

@mcoker
mcoker merged commit dc988cd into patternfly:main Sep 15, 2025
4 checks passed
@mcoker
mcoker deleted the issue-7795 branch September 15, 2025 15:44
@patternfly-build

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 6.3.0-prerelease.69 🎉

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.

Bug - Dual list selector w/ tree view - missing high contrast border on selected element

4 participants