-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Design] create a pie and tree map icon chart for Lens #54291
Comments
Pinging @elastic/kibana-design (Team:Design) |
Time frame is 7.7 |
There are actually three chart types that we can support, if we consider sunburst and donut to be separate types: |
Hey any updates on that one? Who is working on that @ryankeairns ? |
Which charts are we actually designating in the chart switcher for 7.7? |
|
Donut can also be a sunburst isn't it? I would add regular Pie, Donut and treemap where sunburst is a feature inside pie and donut |
To break it down as far as possible, these are all of the possible options for icons we could use in the chart switcher: Single ring
Multiple ring
If you want to make it more explicit when transitioning from single-ring to multi-ring, we can do that. Otherwise, we can make it so that the "rings" option supports one or more rings. |
Hello! 👋 Here are the icons. PreviewIcons DesignI notice all the other icons are inside a frame of 36px x 24px so I followed the rule. They also have a 22px height: In case it's necessary, you can find the Figma project here DownloadThe icons are already optimized and they can be download from the following link: |
@miukimiu Thanks! |
@wylieconlon Feel free to implement the icons into your PR, or just let me know if you'd like me to tackle it. Thanks so much @miukimiu !!! |
We are planning to add pie and treemap charts to Lens shortly.
This issue was the first step in working on chart icon #46287
We will need design's help in creating the new chart icons
cc: @ryankeairns @cchaos
The text was updated successfully, but these errors were encountered: