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

pie color fix & enhancements #2867

Closed
wants to merge 1 commit into from
Closed

pie color fix & enhancements #2867

wants to merge 1 commit into from

Conversation

alexcjohnson
Copy link
Collaborator

  • Fix inheritance of explicit colors by later traces (see Fields missing in fullData #2834 (comment))
  • Allow inheritance (of explicit colors) by earlier traces too - @etpinard I did this by adding a calcInteractions step immediately after calc. In principle I feel like we could move setPositions over to calcInteractions, but not vice versa because setPositions is tied to cartesian subplot. But I didn't touch setPositions in this PR. I also tried putting this code into Pie.plot but this failed because the colors are needed for the legend, which gets drawn before the data.
  • Add layout.piecolorway and layout.extendpiecolors for more control over pie colors (cc @nicolaskruchten @VeraZab)
  • Does not address pie legends with conflicting colors #2858

- fix inheritance of explicit colors by later traces
- allow inheritance (of explicit colors) by earlier traces too
- add `piecolorway` and `extendpiecolors` for more control over pie colors
@nicolaskruchten
Copy link
Contributor

Wow, this seems amazing, thanks Alex! It'll really help make the UX of the editor wayyy better :)

@alexcjohnson
Copy link
Collaborator Author

Closing this PR, will recreate it once #2868 is merged.

@alexcjohnson alexcjohnson deleted the pie-colors branch August 3, 2018 21:42
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.

2 participants