8000 chore(deps): update pf deps by mcoker · Pull Request #6624 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

chore(deps): update pf deps#6624

Merged
mcoker merged 2 commits intopatternfly:v6from
mcoker:issue-6623
May 8, 2024
Merged

chore(deps): update pf deps#6624
mcoker merged 2 commits intopatternfly:v6from
mcoker:issue-6623

Conversation

@mcoker
Copy link
Contributor
@mcoker mcoker commented May 7, 2024

fixes #6623

@patternfly-build
Copy link
Collaborator
patternfly-build commented May 7, 2024

@mcoker mcoker requested a review from srambach May 7, 2024 21:05
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.

One thing - if you look at a full page demo and click the RTL switch, the switches disappear. It looks like it's because the z-index variable var(--pf-v6-global--ZIndex--2xl) is now undefined.

Otherwise, I don't see any other issues.

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.

🚗

@mcoker mcoker merged commit 16d8b6b into patternfly:v6 May 8, 2024
@mcoker
Copy link
Contributor Author
mcoker commented May 8, 2024

The z-index var @srambach mentioned should be fixed once we finish updating docs-framework styles to use tokens

@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.125 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mcoker mcoker mentioned this pull request May 13, 2024
mattnolting pushed a commit to mattnolting/patternfly that referenced this pull request Jun 21, 2024
mattnolting pushed a commit to mattnolting/patternfly that referenced this pull request Jun 21, 2024
@mcoker mcoker deleted the issue-6623 branch January 7, 2025 16:48
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.

3 participants

0