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

Restyle: bar chart #246

Closed
4 tasks
shixiedesign opened this issue Apr 19, 2019 · 3 comments
Closed
4 tasks

Restyle: bar chart #246

shixiedesign opened this issue Apr 19, 2019 · 3 comments

Comments

@shixiedesign
Copy link

shixiedesign commented Apr 19, 2019

New style of Bar chart is below. Reference text in #D6316C for measurements. In general we are thinning the bars to align with the new language's more delicate, refined look & feel.

  • standard bar
  • multi-dimensional bar (grouped bar)
  • stacked bar
  • horizontal bar

(2) Bar charts

@theiliad
Copy link
Member

@shixiedesign Couple of points here:

  1. In stacked bar, how would we add a 1px gap to a linear y-axis? That would introduce some inaccuracy.
  2. We can't have too much fill related logic in elements, since we won't know exactly how users will form their color palettes. If we were to check, that'd introduce lots of additional logic in the code. Seems like some of those items should be enforced by team designers through our design spec documents?

@shixiedesign
Copy link
Author

shixiedesign commented May 6, 2019

@theiliad Again sorry for the late response:

  1. [Stacked bar] Right. Unfortunately, the 1px white line will cause inaccuracy and it's the trade off for accessibility in stacked bar. It's technically not a gap, but a divider line, and for the sake of accuracy, I think it should sit at the base of the top bar, so the bottom bar color will be aligned to true value. We definitely want the tooltip to help with reading exact value (see below img). Eyeballing value would be inaccurate even without the white divider line.
    image

  2. [Multidimensional bar] I believe we agreed to use every nth child to target all the even number bars, and those bars will get a 1px border top with #000000 at 0.3 opacity. Lemme know if this aligns with your understanding.

theiliad added a commit to theiliad/carbon-charts that referenced this issue Sep 19, 2019
@theiliad
Copy link
Member

theiliad commented Jan 9, 2020

These specs were all implemented in Q3 2019. Let me know if anything is missing

In terms of combo charts we should create a separate issue

@theiliad theiliad closed this as completed Jan 9, 2020
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

3 participants