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 #16634 - reset accordion ActiveTab public method #16838

Conversation

kievsash
Copy link
Contributor

@kievsash kievsash commented Nov 25, 2024

Fix for #16634
Problem: In accordion with ngFor-ed tabs when tabs array is changed active tab is still stay active.
Solution: provide public method resetActiveTab() to be able to reset it manually on data change.

Usage example:
image
code

Copy link

vercel bot commented Nov 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
primeng-v18 ❌ Failed (Inspect) Dec 6, 2024 7:54pm
1 Skipped Deployment
Name Status Preview Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Dec 6, 2024 7:54pm

Copy link

vercel bot commented Nov 25, 2024

@kievsash is attempting to deploy a commit to the primetek Team on Vercel.

A member of the Team first needs to authorize it.

@mertsincan mertsincan added the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label Dec 6, 2024
@mertsincan
Copy link
Member

Thanks a lot for your contribution! Could you please remove package-lock.json from PR?

@vercel vercel bot temporarily deployed to Preview – primeng-v18 December 6, 2024 19:52 Inactive
@kievsash kievsash force-pushed the accordion-active-tab-reset-on-tabsdata-change branch from b6b4e76 to de12174 Compare December 6, 2024 19:54
@vercel vercel bot temporarily deployed to Preview – primeng-v18 December 6, 2024 19:54 Inactive
@kievsash
Copy link
Contributor Author

kievsash commented Dec 6, 2024

Hi @mertsincan
package-lock.json is removed from a pr

@mertsincan mertsincan removed the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label Dec 6, 2024
@mertsincan mertsincan merged commit 5f5f28c into primefaces:master Dec 6, 2024
1 of 2 checks passed
@mertsincan
Copy link
Member

Thanks a lot for your contribution!

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.

Component: Accordion do not reset selected active Tab on new dynamic tab data
2 participants