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.

Sign up for GitHub

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

Jump to bottom

AccordionTab error with v-if="false" #1339

Closed
andilem opened this issue Jun 10, 2021 · 0 comments
Closed

AccordionTab error with v-if="false" #1339

andilem opened this issue Jun 10, 2021 · 0 comments
Assignees
@tugcekucukoglu
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Milestone
3.5.1

Comments

@andilem
Copy link

andilem commented Jun 10, 2021

I'm submitting a ...

[x] bug report 
[ ] feature request
[ ] support request

CodeSandbox Case (Bug Reports)
https://codesandbox.io/s/determined-glade-ks1s3?file=/src/App.vue

Current behavior
An AccordionTab with v-if="false" causes a TypeError: child.children.forEach is not a function because child seems to be a string.

Expected behavior
An AccordionTab with v-if="false" is just removed from the DOM.

  • Vue version: 3.1.1
  • PrimeVue version: 3.5.0
  • Browser: [Chrome | ??? ]
@tugcekucukoglu tugcekucukoglu added Type: Bug Issue contains a bug related to a specific component. Something about the component is not working vue2-portable labels Jun 11, 2021
@tugcekucukoglu tugcekucukoglu added this to the 3.5.1 milestone Jun 11, 2021
@tugcekucukoglu tugcekucukoglu self-assigned this Jun 11, 2021
tugcekucukoglu added a commit that referenced this issue Jun 11, 2021
@tugcekucukoglu
Refactor #1339
002b845
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tugcekucukoglu tugcekucukoglu

Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Projects
None yet
Milestone
3.5.1
Development

No branches or pull requests
2 participants
@andilem @tugcekucukoglu