Skip to content

Conversation

@J-Sek
Copy link
Contributor

@J-Sek J-Sek commented Apr 16, 2025

Description

[Enter] should not clear the input.

related to #21221

Markup:

<template>
  <v-app>
    <v-container>
      <pre>value: {{ value }} | {{ typeof value }}</pre>
      <v-date-input v-model="value" />
    </v-container>
  </v-app>
</template>

<script setup>
  import { ref } from 'vue'
  const value = ref()
</script>

@J-Sek J-Sek self-assigned this Apr 16, 2025
@J-Sek J-Sek added T: bug Functionality that does not work as intended/expected C: VDateInput labels Apr 16, 2025
@J-Sek J-Sek force-pushed the fix/vdateinput-typing-without-picker branch from d6646c6 to bc66b7e Compare April 16, 2025 08:06
Copy link
Member

@ikushum ikushum left a comment

Choose a reason for hiding this comment

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

Looks good

@ikushum
Copy link
Member

ikushum commented Apr 16, 2025

Looks like there are lint issues

@J-Sek J-Sek requested a review from johnleider April 16, 2025 17:33
@ikushum
Copy link
Member

ikushum commented May 11, 2025

This bug is quite obvious and annoying. What do we need to merge this? @J-Sek @johnleider

@johnleider
Copy link
Member

It was approved, author never merged.

@J-Sek J-Sek force-pushed the fix/vdateinput-typing-without-picker branch 2 times, most recently from 0c858fd to 19f2917 Compare May 12, 2025 18:38
@J-Sek J-Sek force-pushed the fix/vdateinput-typing-without-picker branch from 19f2917 to eba09cf Compare May 12, 2025 18:42
@J-Sek
Copy link
Contributor Author

J-Sek commented May 12, 2025

I don't like the fact the tests are a bit brittle... needed to move the new one as first, otherwise it fails. But getting the fix out is more important atm.

@J-Sek J-Sek merged commit 99331d6 into vuetifyjs:master May 12, 2025
10 checks passed
@J-Sek J-Sek deleted the fix/vdateinput-typing-without-picker branch May 12, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: VDateInput T: bug Functionality that does not work as intended/expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants