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

Recursive error when using a ref anywhere in the canvas when it changes on a click event handler inside a v-for #649

Open
5 tasks done
franciscohermida opened this issue Apr 16, 2024 · 1 comment
Labels
pending-triage Ticket is pending to be prioritised v4

Comments

@franciscohermida
Copy link

franciscohermida commented Apr 16, 2024

Describe the bug

Doing:

  • v-for rendering 3 boxes
  • click event handler to set currentItem ref for each box
  • using the currentItem ref elsewhere in the canvas

Expecting:
No recursive error

Happening:
recursive error because click handler is called again when ref is being used elsewhere in the canvas

Reproduction

https://stackblitz.com/edit/tresjs-basic-bskmaj?file=src%2Fcomponents%2FBox.vue

Steps to reproduce

Click on each of the boxes and check console

System Info

No response

Used Package Manager

npm

Code of Conduct

@alvarosabu
Copy link
Member

Hi @franciscohermida thanks for opening this, I think this will be fixed with #529 on the v4 release.

We are fine-tuning the PR and we expect to merge it soon, I will let you know so you can test it with the next channel release.

@alvarosabu alvarosabu added bug Something isn't working pending-triage Ticket is pending to be prioritised v4 and removed bug Something isn't working labels Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending-triage Ticket is pending to be prioritised v4
Projects
None yet
Development

No branches or pull requests

2 participants