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

Show color in ColorInputComponent, just like in ColorFieldComponent #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gartner
Copy link

@gartner gartner commented Oct 19, 2020

Add a square, showing the current color of the ColorInputComponent, just like it is shown in ColorFieldComponent.

@Chrisgolden77
Copy link

Any news on if this can be merged? it would be a great feature

@gartner
Copy link
Author

gartner commented Jan 25, 2021

I have published a fork of this package, called @palustris/react-admin-color-input. My package contains this change, but also a few others, ie. easier navigation using the keyboard.

@Chrisgolden77
Copy link

Thanks @gartner

@notz
Copy link

notz commented Feb 2, 2021

@gartner Thank you very much for your updated version. Maybe you can add support for empty values in ColorField. Currently it crashes because of ColorDot "TypeError: Cannot read property '0' of undefined"

@gartner
Copy link
Author

gartner commented Feb 2, 2021

@gartner Thank you very much for your updated version. Maybe you can add support for empty values in ColorField. Currently it crashes because of ColorDot "TypeError: Cannot read property '0' of undefined"

I will have a look at it. Added an issue for it here gartner#1

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

Successfully merging this pull request may close these issues.

3 participants