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

[Bug Report] V-Select name prop does not work anymore #10152

Closed
Kotoriii opened this issue Jan 8, 2020 · 2 comments · Fixed by skyYaga/skdvin-webapp#21 or anyulled/reactivewebflux#5
Closed

[Bug Report] V-Select name prop does not work anymore #10152

Kotoriii opened this issue Jan 8, 2020 · 2 comments · Fixed by skyYaga/skdvin-webapp#21 or anyulled/reactivewebflux#5
Assignees
@johnleider
Labels
C: VSelect VSelect S: has PR The issue has a pending Pull Request T: regression Something that used to work but we broke
Milestone
v2.2.x

Comments

@Kotoriii
Copy link

Kotoriii commented Jan 8, 2020 •
edited
Loading

Environment

Vuetify Version: 2.2.3
Last working version: 2.1.9
Vue Version: 2.6.10
Browsers: Chrome 79.0.3945.88
OS: Mac OS 10.14.6

Steps to reproduce

Add a name prop to a v-select

Expected Behavior

name prop appears in html element inspector inside input element.

Actual Behavior

name prop is not visible in input element.

Reproduction Link

https://codepen.io/kotoriii/pen/Jjopajz

Other comments

Still works in v-autocomplete
@ghost ghost added the S: triage label Jan 8, 2020
@MajesticPotatoe
Copy link
Member

#9598
more specifically
Input L415
and
Comboboc L72

@KaelWD
Copy link
Member

KaelWD commented Jan 8, 2020

@Mouquette
image

@johnleider johnleider added T: bug Functionality that does not work as intended/expected and removed S: triage labels Jan 9, 2020
cyruscollier added a commit to cyruscollier/vuetify that referenced this issue Jan 25, 2020
@cyruscollier
fix(vselect): restore name prop behavior
08bfe92 
Fixes missing name attribute on inner input element by passing it manually from component
attributes. Includes unit tests on VSelect and VCombobox to verify bugfix.

fix vuetifyjs#10152
@cyruscollier cyruscollier mentioned this issue Jan 25, 2020
Merged
8 tasks
@johnleider johnleider added T: regression Something that used to work but we broke and removed T: bug Functionality that does not work as intended/expected labels Jan 27, 2020
@johnleider johnleider added this to the v2.2.x milestone Jan 27, 2020
@johnleider johnleider added the S: has PR The issue has a pending Pull Request label Jan 27, 2020
@johnleider johnleider self-assigned this Jan 27, 2020
@johnleider johnleider added C: VSelect VSelect and removed C: VCombobox VCombobox C: VInput VInput labels Jan 27, 2020
@snyk-bot snyk-bot mentioned this issue Feb 21, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Mar 14, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnleider johnleider

Labels
C: VSelect VSelect S: has PR The issue has a pending Pull Request T: regression Something that used to work but we broke
Projects
None yet
Milestone
v2.2.x
4 participants
@Kotoriii @johnleider @KaelWD @MajesticPotatoe