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

feat(ui/otp-Input): add OTP input component #1614

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

Aybrea
Copy link
Collaborator

@Aybrea Aybrea commented May 5, 2024

Checklist

List of tasks you have already done and plan to do.

  • Fix linting errors
  • Tests have been added / updated (or snapshots)

Change information

Add OTP input component

Issues

close #1603

Copy link

vercel bot commented May 5, 2024

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

Name Status Preview Comments Updated (UTC)
varlet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2024 2:19pm

Copy link

vercel bot commented May 5, 2024

@Aybrea is attempting to deploy a commit to the varletjs Team on Vercel.

A member of the Team first needs to authorize it.

packages/varlet-ui/src/otp-input/OtpInput.vue Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/OtpInput.vue Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/OtpInput.vue Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/docs/zh-CN.md Outdated Show resolved Hide resolved
packages/varlet-ui/src/utils/elements.ts Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/OtpInput.vue Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/docs/en-US.md Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/props.ts Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/example/index.vue Outdated Show resolved Hide resolved
packages/varlet-ui/src/otp-input/docs/zh-CN.md Outdated Show resolved Hide resolved
@haoziqaq haoziqaq changed the title feat(ui/otpInput): add OTP input component feat(ui/otp-Input): add OTP input component May 6, 2024
@chouchouji
Copy link
Member

chouchouji commented May 9, 2024

要考虑form的只读和禁用,参考 input 组件

@YHoney7
Copy link
Member

YHoney7 commented May 30, 2024

2024-05-31.00.05.36.mov

发现了个问题, 如果用Tab键切换聚焦到第二输入框, 再次输入的话, 聚焦并不是从第一个输入框开始的

@Aybrea
Copy link
Collaborator Author

Aybrea commented Jun 1, 2024

2024-05-31.00.05.36.mov
发现了个问题, 如果用Tab键切换聚焦到第二输入框, 再次输入的话, 聚焦并不是从第一个输入框开始的

已修正,感谢指出

@chouchouji
Copy link
Member

chouchouji commented Jun 2, 2024

20240602162855_rec_.mp4

这里有点问题,第一次删除的时候,就直接切换到下一个能删除的地方了,第二个点击删除键,还是停留在删除的那个输入框里

@chouchouji
Copy link
Member

20240602162855_rec_.mp4
这里有点问题,第一次删除的时候,就直接切换到下一个能删除的地方了,第二个点击删除键,还是停留在删除的那个输入框里

已修复

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.

[Feature]: New component OtpInput
4 participants