Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(accordion): respect presence of expanded in html #2308

Merged
merged 8 commits into from Jan 13, 2023

Conversation

bennypowers
Copy link
Member

What I did

test and ensure that when expanded is present on an accordion header in HTML, it's associated panel will expand on the page.

Testing instructions

unit tests, check the DP, play with the demos

@changeset-bot
Copy link

changeset-bot bot commented Jan 12, 2023

🦋 Changeset detected

Latest commit: ea04fc1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/pfe-accordion Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@changeset-bot
Copy link

changeset-bot bot commented Jan 12, 2023

🦋 Changeset detected

Latest commit: 3dd79c8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/pfe-accordion Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added demo Updating demo pages functionality Functionality, typically pertaining to the JavaScript. tests Related to testing labels Jan 12, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 2023

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 5ff1f9d
😎 Deploy Preview https://deploy-preview-2308--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added the AT passed Automated testing has passed label Jan 12, 2023
@bennypowers bennypowers enabled auto-merge (rebase) January 12, 2023 13:55
Copy link
Collaborator

@brianferry brianferry left a comment

Choose a reason for hiding this comment

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

Looks good to me code / testing wise, resolve when all testing is completed

@bennypowers bennypowers merged commit db95a1a into main Jan 13, 2023
8 checks passed
@bennypowers bennypowers deleted the fix/accordion/expand-attr-initial branch January 13, 2023 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT passed Automated testing has passed demo Updating demo pages functionality Functionality, typically pertaining to the JavaScript. ready to merge tests Related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants