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.

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

[Bug Report] The loading prop doesn't work on v-selects with the solo prop #2399

Closed
stoked74 opened this issue Nov 1, 2017 · 3 comments
Closed
Assignees
Labels
T: bug Functionality that does not work as intended/expected T: enhancement Functionality that enhances existing features
Milestone

Comments

@stoked74
Copy link

stoked74 commented Nov 1, 2017

Versions and Environment

Vuetify: v0.17.0-beta.1
Vue: v2.5.2
Browsers: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.75 Safari/537.36
OS: Mac OSX

Steps to reproduce

Create an async v-select with the solo prop and the loading prop. Set the loading prop to true.

Expected Behavior

I would expect the loader to still be visible even though this is a solo v-select. I think there's definitely a need/use case having solo v-selects operate asynchronously with a visible loader.

Actual Behavior

When the :loading prop is true the loader isn't visible on solo v-selects

Reproduction Link

https://codepen.io/smitchell/pen/NwGMvK?editors=1010

Other comments

I'm really not sure if this a bug or a feature request but I think the loader should operate consistently across all v-select variants.

@jacekkarczmarczyk
Copy link
Member

Same for text fields
Simplified codepen: https://codepen.io/anon/pen/dZYjJN?editors=1010

@nekosaur nekosaur added the pending review The issue is still pending disposition label Nov 5, 2017
@johnleider johnleider added T: bug Functionality that does not work as intended/expected T: enhancement Functionality that enhances existing features and removed pending review The issue is still pending disposition labels Nov 6, 2017
@johnleider johnleider added this to the v1.1.x milestone Nov 6, 2017
@johnleider johnleider self-assigned this Nov 6, 2017
@KaelWD
Copy link
Member

KaelWD commented Nov 10, 2017

#2273 would probably fix this

@johnleider
Copy link
Member

Loaders for everyone!
l test_8088_ 7

@johnleider johnleider mentioned this issue May 6, 2018
42 tasks
johnleider added a commit that referenced this issue May 6, 2018
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T: bug Functionality that does not work as intended/expected T: enhancement Functionality that enhances existing features
Projects
None yet
Development

No branches or pull requests

5 participants