-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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
[Feature Request] disable prop for v-form that disables all child input elements #1707
Comments
I really like this idea. What do you think @nekosaur ? |
I like the idea, but I'll have to test it out. I'm thinking we'll have some issues with mutating props if we programatically set disabled from VForm while there's an explicit disabled prop on an input inside it. |
I might be a good idea to add also We may also want to wait for this vuejs/vue#6097. With this PR we'll be able to create a |
The only problem with that @jacekkarczmarczyk is that injections are not reactive. I'd imagine you'd want to re-activate the form at some point no? |
I would also vote for Ideally, inputs with content would be read-only, while buttons and other inputs disabled. |
Moving to 1.1 since it already includes a planned refactor of input components |
Pushing to 1.2 so we can actually get 1.1 out this year. I did a proof of concept with #3969 |
Any chance, that this feature will fall to 1.3.0 ? |
This is unfortunately low on triage and did not make it for 1.3. Going to add a help wanted tag if anyone wants to PR. |
It seems that this feature can be created with reference to Themeable. |
If do not use mixin , there is also a way to provide from v-form. |
We still intend to implement this but it did not make the cut for v2.0 |
At work we are about to start a new project with vuetify. We are wireframing at the moment, but if this has not been fixed by the time we start programming, I'll see if I can squeeze it into a sprint to add |
I think the quickest way to do that is by to implement fieldset and apply the disabled styles by class like |
Do you have any examples on how to do that? I am looking for a quick way to conditionally disable multiple forms and I could not get fieldset to work. |
What is the status of this feature request? Will it be implemented in the next version? |
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
A disabled VForm now disables all child inputs fixes vuetifyjs#1707
fixes #1707 * feat(VForm): Add form level disabled prop A disabled VForm now disables all child inputs * Fixed formatting based on review * Better wording for disabled prop documentation
fixed in #10325 |
What will it allow you to do that you can't do today?
Disable all fields in a form with one prop
How will it make current work-arounds straightforward?
Remove need to add a disabled prop to every item on a form.
What potential bugs and edge cases does it help to avoid?
Forgetting to add a disabled prop to a given form element
The text was updated successfully, but these errors were encountered: