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.

Sign up for GitHub

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

Jump to bottom

fix(VTimePickerControls): disable in disabled forms #20983

Closed
wants to merge 3 commits into from
Closed

fix(VTimePickerControls): disable in disabled forms #20983

wants to merge 3 commits into from

Conversation

mtdvlpr
Copy link
Contributor

@mtdvlpr mtdvlpr commented Feb 13, 2025

Description

Markup:

<template>
    <v-form disabled>
        <v-time-picker />
    </v-form>
</template>

@KaelWD KaelWD force-pushed the master branch 2 times, most recently from 383dac4 to 812af2e Compare February 14, 2025 06:34
@MajesticPotatoe MajesticPotatoe added T: bug Functionality that does not work as intended/expected C: VTimePicker labels Feb 14, 2025
@KaelWD
Copy link
Member

KaelWD commented Feb 17, 2025

This does nothing as VTimePicker is not form aware. It can be revisited if we ever do a VTimeInput.

@KaelWD KaelWD closed this Feb 17, 2025
@mtdvlpr
Copy link
Contributor Author

mtdvlpr commented Feb 17, 2025

You're already working on it: #19709, but it doesn't include this fix yet.

@mtdvlpr mtdvlpr mentioned this pull request Feb 17, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
C: VTimePicker T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mtdvlpr @KaelWD @MajesticPotatoe