Skip to content

Comments

Implemented dual-y axis on line chart#185

Open
andy-lee-eng wants to merge 59 commits intodevelopfrom
feature/dual-y-axis
Open

Implemented dual-y axis on line chart#185
andy-lee-eng wants to merge 59 commits intodevelopfrom
feature/dual-y-axis

Conversation

@andy-lee-eng
Copy link
Collaborator

Uses a decorated chart to fill in the 5th grid column with a second y-axis, and render both sets of series with their corresponding scales.

Uses a decorated chart to fill in the 5th grid column with a second y-axis, and render both sets of series with their corresponding scales.
Splitting by name is necessary when we have a split-by value, because there are multiple series per mainValue, and we can't identify them by index.

Color is applied when there isn't a split-by, but can't be when there is because there are several colours.
The Zoomable component was replacing the decorator function, so we need to chain them
@andy-lee-eng andy-lee-eng force-pushed the feature/dual-y-axis branch from 0f6ac23 to 816ee7c Compare May 3, 2019 16:14
Copy link
Collaborator

@mhooper-scottlogic mhooper-scottlogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks really good.

image

mhooper-scottlogic and others added 18 commits May 7, 2019 17:29
Correctly read and generate boolean values for Arrow format
Update results.json to reflect Scatter name change
use fade transition, using pre-existing map to define positioning.
sc1f and others added 30 commits May 9, 2019 11:09
Refactor Javascript API +add documentation
…umber_formats

Fix the format of long numbers
…p_enhancements

Feature/treemap enhancements
Performance overhaul for `table.update()`
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.

6 participants