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] Circular Progress Bar #5874

Closed
amroessam opened this issue Dec 12, 2018 · 2 comments · Fixed by #5875
Closed

[Bug Report] Circular Progress Bar #5874

amroessam opened this issue Dec 12, 2018 · 2 comments · Fixed by #5875
Labels
good first issue A quick-win fix that would be great for new contributors T: enhancement Functionality that enhances existing features

Comments

@amroessam
Copy link

Versions and Environment

Vuetify: 1.3.13
Vue: 2.5.21
Browsers: Chrome 69.0.3497.92
OS: Windows 10

Steps to reproduce

add width attribute to v-progress-circular

Expected Behavior

No warnings in the console

Actual Behavior

Works, but logs an error in the console
Invalid prop: type check failed for prop "width"

Reproduction Link

https://codepen.io/anon/pen/xmGBLN

Other comments

Can make it work if you bind it to a data property, but this is unnecessary since it should convert the string to a number

@jacekkarczmarczyk jacekkarczmarczyk added T: enhancement Functionality that enhances existing features good first issue A quick-win fix that would be great for new contributors labels Dec 12, 2018
@dsseng
Copy link
Contributor

dsseng commented Dec 12, 2018

Workaround: :width="10"

@amroessam
Copy link
Author

Yeah, but again you're binding to a value, makes no sense, should just convert the string to int when passing to the css

MajesticPotatoe pushed a commit to dsseng/vuetify that referenced this issue Dec 12, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Dec 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue A quick-win fix that would be great for new contributors T: enhancement Functionality that enhances existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants