Skip to content

fix(expandable section): update to use <Button> for toggle#10891

Merged
dlabaj merged 3 commits intopatternfly:mainfrom
tlabaj:exp_section_btn
Sep 5, 2024
Merged

fix(expandable section): update to use <Button> for toggle#10891
dlabaj merged 3 commits intopatternfly:mainfrom
tlabaj:exp_section_btn

Conversation

@tlabaj
Copy link
Contributor

@tlabaj tlabaj commented Aug 28, 2024

What: Closes #10878

@tlabaj tlabaj requested review from a team, kmcfaul, mattnolting and wise-king-sullyman and removed request for a team August 28, 2024 19:35
@tlabaj tlabaj linked an issue Aug 28, 2024 that may be closed by this pull request
Copy link
Contributor

@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

Some cypress tests just need to be updated, most likely due to the expandable-section__toggle class now being a div that contains a button

@tlabaj tlabaj marked this pull request as draft August 29, 2024 14:53
@tlabaj tlabaj marked this pull request as ready for review August 30, 2024 13:46
@patternfly-build
Copy link
Collaborator

patternfly-build commented Aug 30, 2024

Copy link
Contributor

@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

LGTM!

@dlabaj dlabaj merged commit fd9ce14 into patternfly:main Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expandable section - update to use <Button> for toggle

6 participants