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

[Feature] Disable/hide numeric fields #94

Closed
derekphilipau opened this issue Oct 28, 2017 · 3 comments
Closed

[Feature] Disable/hide numeric fields #94

derekphilipau opened this issue Oct 28, 2017 · 3 comments

Comments

@derekphilipau
Copy link

Just as you have the "disableAlpha" property to remove alpha, I think it would be good to have a "disableFields" property to remove the fields, e.g. <div class="vc-chrome-fields-wrap" v-if="!disableFields">

For many users, they are not geeks and don't care about numeric values of the color!

Great work, very useful, thanks!

@linx4200
Copy link
Collaborator

@derekphilipau Thanks for your feedback. I think it's resonable, and we will put it into our todo list and implement it maybe in the next version.

Of cource, contribution is welcome! 😄😄😄

maxirozay added a commit to maxirozay/vue-color that referenced this issue Nov 2, 2017
maxirozay added a commit to maxirozay/vue-color that referenced this issue Nov 2, 2017
linx4200 added a commit that referenced this issue Nov 6, 2017
@derekphilipau
Copy link
Author

My use case is probably too specific, I just want simplest RGB value possible. Just in case someone is interested I modified the Chrome component to: a) not show an "active-color" swatch, b) not show alpha, c) button on parent controls show/hide of color component, d) button background/text color dependent upon color value. vue-color is really nice and easy to customize, thanks!
screen shot 2017-11-17 at 10 04 06

@linx4200
Copy link
Collaborator

@derekphilipau In your case, I think using Saturation and Hue direactly to compose your component would be a better option.

And we released 2.4.3 just now. In this new release we added disableFields props to help hide the numberic fields. Check this pr for more info.

Thanks for your feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants