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

stack order comparator #1642

Merged
merged 6 commits into from
May 27, 2023
Merged

Conversation

mbostock
Copy link
Member

@mbostock mbostock commented May 26, 2023

Fixes #1195. Fixes the stack part of #1613.

@mbostock mbostock requested a review from Fil May 26, 2023 22:57
@mbostock mbostock marked this pull request as ready for review May 26, 2023 22:57
@mbostock mbostock force-pushed the mbostock/stack-descending-order-shorthand branch from 792f0ec to 6efe4b8 Compare May 27, 2023 17:18
@mbostock mbostock merged commit 20eeb56 into main May 27, 2023
@mbostock mbostock deleted the mbostock/stack-descending-order-shorthand branch May 27, 2023 17:21
chaichontat pushed a commit to chaichontat/plot that referenced this pull request Jan 14, 2024
* stack order comparator

* stack descending shorthand

* transpose orderZDomain

* more inlining

* inline options

* better comparator type
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.

The stack order option should support a comparator
2 participants