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

Add Color Palette Display component #3575

Closed
miukimiu opened this issue Jun 8, 2020 · 0 comments · Fixed by #3865
Closed

Add Color Palette Display component #3575

miukimiu opened this issue Jun 8, 2020 · 0 comments · Fixed by #3865
Assignees

Comments

@miukimiu
Copy link
Contributor

miukimiu commented Jun 8, 2020

EuiColorPaletteDisplay

This issue is to track the EuiColorPaletteDisplay which is a follow up on #2795 (Add Color Palette Picker component and widget).

The idea of the EuiColorPaletteDisplay is that we can use the palettes displays outside of the EuiColorPaletterPicker. So we can create this type of UIs:

80425885-93982780-88b2-11ea-8ccf-0eeeccd3ef68 (2)

Screenshot 2020-06-08 at 12 50 32

Development

The color palette picker uses two functions to generate the displays inside the inputs: getLinearGradient and getFixedLinearGradient.

We can reuse these two functions to generate the palette displays. But a few things must be addressed:

  • Can these two functions be converted in just one?
  • Can we reuse/share code of the EuiColorStops?
  • Both functions should receive stops. Right now just the getLinearGradient is able to generate a palette based on stops.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants