8000 fix(accordion): fix nested accordion bug by mcoker · Pull Request #7752 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

fix(accordion): fix nested accordion bug#7752

Merged
mcoker merged 2 commits intopatternfly:mainfrom
mcoker:issue-7736
Aug 27, 2025
Merged

fix(accordion): fix nested accordion bug#7752
mcoker merged 2 commits intopatternfly:mainfrom
mcoker:issue-7736

Conversation

@mcoker
Copy link
Contributor
@mcoker mcoker commented Aug 21, 2025

fixes #7736

@patternfly-build
Copy link
Collaborator
patternfly-build commented Aug 21, 2025

@mcoker mcoker force-pushed the issue-7736 branch 2 times, most recently from 3d26565 to 8c3fb1c Compare August 26, 2025 00:01
@mcoker mcoker marked this pull request as ready for review August 26, 2025 01:11
@mcoker mcoker requested a review from srambach August 26, 2025 01:11
@mcoker mcoker closed this Aug 26, 2025
@mcoker mcoker reopened this Aug 26, 2025
8000
--#{$accordion}__expandable-content--FontSize: var(--pf-t--global--font--size--body--default);
--#{$accordion}__expandable-content--m-fixed--MaxHeight--base: 0;
--#{$accordion}__expandable-content--m-fixed--MaxHeight: #{pf-size-prem(150px)};
--#{$accordion}__expandable-content--MarginBlockEnd: 0;
Copy link
Member

Choose a reason for hiding this comment

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

This line is a repeat of line 43?

@mcoker mcoker requested a review from srambach August 27, 2025 19:20
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 d7e28f1 into patternfly:main Aug 27, 2025
4 checks passed
@mcoker mcoker deleted the issue-7736 branch August 27, 2025 20:42
@patternfly-build
Copy link
Collaborator

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

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 - [Accordion] - In Nested Accordions, the inner Accordion is always open

3 participants

0