Skip to content

fix(accordion): transition display instead of hidden attr#7513

Merged
mcoker merged 1 commit into
patternfly:mainfrom
mcoker:issue-7512
May 7, 2025
Merged

fix(accordion): transition display instead of hidden attr#7513
mcoker merged 1 commit into
patternfly:mainfrom
mcoker:issue-7512

Conversation

@mcoker

@mcoker mcoker commented May 7, 2025

Copy link
Copy Markdown
Contributor

fixes #7512

@patternfly-build

patternfly-build commented May 7, 2025

Copy link
Copy Markdown
Collaborator

@thatblindgeye thatblindgeye left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Linked to React locally and things look good. There's still that visual thing where clicking open one toggle when another toggle is already open has a slight "snap", but if that's still fine from design per convo yesterday then React should be good without any further work (worth double checking in a core bump PR, though)

@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.

Looks good

@mcoker mcoker merged commit 75217be into patternfly:main May 7, 2025
4 checks passed
@mcoker mcoker deleted the issue-7512 branch May 7, 2025 21:25
@patternfly-build

Copy link
Copy Markdown
Collaborator

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

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.

Accordion - toggle display instead of hidden

4 participants