Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(PinInput): error throw from resetPlaceholder's nextTick if component not in the DOM anymore #1606

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

hartbit
Copy link
Contributor

@hartbit hartbit commented Feb 3, 2025

This resolves the issue I discovered here: #1605
I have a reproduction example app (https://stackblitz.com/edit/bhw6xwfr-hcdmactw?file=src%2FApp.vue,package.json) but I failed to reproduce it in the unit tests :(

Copy link
Member

@zernonia zernonia left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @hartbit ! 🙏🏼

@zernonia zernonia merged commit 453437b into unovue:main Feb 4, 2025
3 checks passed
Copy link

pkg-pr-new bot commented Feb 4, 2025

Open in Stackblitz

npm i https://pkg.pr.new/radix-vue@1606

commit: 0a1f18b

@hartbit
Copy link
Contributor Author

hartbit commented Feb 4, 2025

@zernonia This probably needs to be cherry picked on v2, right ?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: PinInputInput Cannot read properties of null (reading 'value') in nextTick
2 participants