Skip to content

feat!: add useVueOTPContext, forwardRef with $el#19

Merged
wobsoriano merged 14 commits intowobsoriano:mainfrom
sadeghbarati:feat/update
Nov 1, 2025
Merged

feat!: add useVueOTPContext, forwardRef with $el#19
wobsoriano merged 14 commits intowobsoriano:mainfrom
sadeghbarati:feat/update

Conversation

@sadeghbarati
Copy link
Copy Markdown
Contributor

@wobsoriano Hello Robert sry for direct mention and big PR,

We are using this lib in v4 registry of shadcn-vue thanks for this port
We need to match our OTP component with main shadcn-ui which uses context

Here the list of changes:

  • tsdown
  • provide context
  • defaultValue prop
  • forwardRef with $el instead of ref so user could retrieves the underlying DOM with unrefElement
  • update workspace, website, use tailwindcss v4 and latest shadcn-vue components registry files
  • vue-tsc v3

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 28, 2025

@sadeghbarati is attempting to deploy a commit to the Robert Soriano's projects Team on Vercel.

A member of the Team first needs to authorize it.

move attrs outside of the computed and use in template since $attrs are readonly and must not be reactive
@wobsoriano
Copy link
Copy Markdown
Owner

Thanks so much brother! I will review it over the weekend 👍🏼

Appreciate it

add _beforeInputListener for preventing selectionchange conflict with input event (thanks grok :)

removed use-previous and use vueuse function
Copy link
Copy Markdown
Owner

@wobsoriano wobsoriano left a comment

Choose a reason for hiding this comment

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

This looks good. I'm merging this.

I'll also make another update soon to match with the changes in the input-otp repo

thanks once again!

@wobsoriano wobsoriano merged commit 15ed989 into wobsoriano:main Nov 1, 2025
1 check failed
@sadeghbarati sadeghbarati deleted the feat/update branch November 2, 2025 02:19
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.

2 participants