-
Notifications
You must be signed in to change notification settings - Fork 609
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
feat(Range): new component #290
Conversation
Someone is attempting to deploy a commit to the Nuxt Labs Team on Vercel. A member of the Team first needs to authorize it. |
✅ Live Preview ready!
|
✅ Live Preview ready!
|
Hello @TomSmith27, Thank you for starting this component! For the styling, I would prefer something like https://ui.shadcn.com/docs/components/slider with a touch of primary color, similar to the Toggle component. Also, all the styles should be defined in the Let me know when you're close, I'll tweak the styles if needed to keep consistency between all the components 😊 |
I'm wondering if it wouldn't be better to rename the component to https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/range#additional_attributes |
For the styling, it might be easier to style it with |
Yep, i was planning to add those, was just wanting to make sure you still wanted this before doing the extra bits. |
I also like the shadcn version so i will attempt to style it like that |
type: [String, Number], | ||
default: '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think modelValue
should be a Number with a default value of 0, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep have fixed
… into 175-slider-component
I have now updated it to be based on this design https://www.material-tailwind.com/docs/react/slider, let me know your thoughts Thanks :) |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
I've just made some updates to your component, let me know what you think 😊 |
.prettierrc.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the UI does not use a prettier, it uses Eslint ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we can let it inside the project, so people who have the Prettier extension on VSCode can contribute easily?
Yep all looks great thanks for taking the time to review it! |
Co-authored-by: Tom Smith <tom.smith2711@gmail.com> Co-authored-by: Benjamin Canac <canacb1@gmail.com> Co-authored-by: Tom Smith <tom.smith@qunifi.com>
Co-authored-by: Tom Smith <tom.smith2711@gmail.com> Co-authored-by: Benjamin Canac <canacb1@gmail.com> Co-authored-by: Tom Smith <tom.smith@qunifi.com>
Resolves #175
I saw that in #175 there was a feature request for a slider component, and i am looking to use NuxtLabs in an upcoming project, and i will need the slider. Can you let me know if this is the kind of component you are looking for, Its copied from DaisyUI https://daisyui.com/components/range/ with some tweaks to make it work in NuxtLabs.
The code is not finished but i wanted to check before tidying it all up
@benjamincanac I believe you will be the best person to look at this