feat(AuthForm): allow all input types #5565
Conversation
commit: |
|
Would it instead be possible to reuse It might not make sense for all of them but I can certainly see using stuff like date when using the |
I think ui/src/runtime/components/Input.vue Line 23 in 8dc6b22 |
|
@benjamincanac Is there a reason why the input types are selectively picked or can we allow any input type? |
|
I think by default we only supported |
J-Michalek
changed the title
|
|
||
| - `name?: string`{lang="ts-type"} | ||
| - `type?: 'text' | 'password' | 'email' | 'number' | 'checkbox' | 'select' | 'otp'`{lang="ts-type"} | ||
| - `type?: 'text' | 'password' | 'email' | 'number' | 'tel' | 'checkbox' | 'select' | 'otp'`{lang="ts-type"} |
There was a problem hiding this comment.
@benjamincanac How will we handle supporting all input types in the docs? Do we list all of them or add something like InputHTMLAttributes['type']?
There was a problem hiding this comment.
I guess you can use | InputHTMLAttributes['type'] yes!
| - `name?: string`{lang="ts-type"} | ||
| - `type?: 'text' | 'password' | 'email' | 'number' | 'checkbox' | 'select' | 'otp'`{lang="ts-type"} | ||
| - `name: string`{lang="ts-type"} | ||
| - `type: 'checkbox' | 'select' | 'otp' | 'InputHTMLAttributes['type']'`{lang="ts-type"} |
There was a problem hiding this comment.
I removed the question marks because both props are required and the description below also stated that the type must be included.
|
This should be ready. |
4 participants
🔗 Linked issue
Resolves #5564
❓ Type of change
📚 Description
The telephone field is often used in registration forms - makes sense to support it.
📝 Checklist