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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
new(demo/xychart): add AreaStack, add stackOffset variable for stacks #1020
Conversation
@@ -720,6 +792,11 @@ export default function ExampleControls({ children }: ControlsProps) { | |||
input[type='radio'] { | |||
height: 10px; | |||
} | |||
.pattern-lines { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this pattern-only svg currently has a non-zero height on /xychart
, this fixes it.
onChange={() => setRenderBarStackOrGroup('bar')} | ||
onChange={() => { | ||
if (renderAreaLineOrStack === 'areastack') { | ||
setRenderAreaLineOrStack('none'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
previously I disabled updating renderAreaLineOrStack
depending on the value of renderBarStackOrGroup
, I think it's better UX if I just update the value of renderAreaLineOrStack
so it's valid.
40c3855
to
eb30e36
Compare
a078919
to
96284ef
Compare
Pull Request Test Coverage Report for Build 513505979
馃挍 - Coveralls |
馃殌 Enhancements
This updates the
/xychart
demo to includeAreaStack
added in #1019 (see images there), and also adds a toggle forstackOffset
for bothAreaStack
andBarStack
.@hshoff @kristw