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

Radar #581

Merged
merged 96 commits into from
May 8, 2020
Merged

Radar #581

merged 96 commits into from
May 8, 2020

Conversation

ilariaventurini
Copy link
Contributor

@ilariaventurini ilariaventurini commented Apr 17, 2020

Core deploy: https://deploy-preview-581--carbon-charts-core.netlify.app/?path=/story/radar--radar

Closes #562
Closes #256

Demo screenshot or recording

Design:
image

Review checklist (for reviewers only)

  • Demos all features
  • Documented/annotated
  • Matches UI/UX specs
  • Meets the code style guide
  • Accessible
  • Mobile first (responsive)
  • RTL support (bidirectional text)
  • Performant (limited bloat)

@netlify
Copy link

netlify bot commented Apr 17, 2020

Deploy preview for carbon-charts-vue ready!

Built with commit 7cdc17c

https://deploy-preview-581--carbon-charts-vue.netlify.app

@netlify
Copy link

netlify bot commented Apr 17, 2020

Deploy preview for carbon-charts-angular ready!

Built with commit 7cdc17c

https://deploy-preview-581--carbon-charts-angular.netlify.app

@netlify
Copy link

netlify bot commented Apr 17, 2020

Deploy preview for carbon-charts-react ready!

Built with commit 7cdc17c

https://deploy-preview-581--carbon-charts-react.netlify.app

@natashadecoste natashadecoste mentioned this pull request Apr 21, 2020
1 task
@ilariaventurini
Copy link
Contributor Author

At the moment the mapping of the column name of the dataset to the angle and value is hardcoded in the code.
I will add a configuration object in radar configuration to make the graph more generic.
Then I will also add some other examples.

theiliad
theiliad previously approved these changes May 5, 2020
Copy link
Member

@theiliad theiliad left a comment

Choose a reason for hiding this comment

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

LGTM

packages/core/src/styles/graphs/_radar.scss Outdated Show resolved Hide resolved
packages/core/src/styles/graphs/_radar.scss Outdated Show resolved Hide resolved
packages/core/src/styles/graphs/_radar.scss Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
packages/core/src/components/graphs/radar.ts Outdated Show resolved Hide resolved
@theiliad
Copy link
Member

theiliad commented May 8, 2020

only issue I find is when height is set to 100 the chart disappears, which I don't think is too much of a bug, at least for now

image

@theiliad theiliad merged commit f9cccca into carbon-design-system:master May 8, 2020
@theiliad theiliad deleted the radar branch May 8, 2020 15:12
@theiliad
Copy link
Member

theiliad commented May 8, 2020

I think wrappers are missing

@ilariaventurini
Copy link
Contributor Author

@theiliad You are right, in that case there is not enough space to draw the radar. I think that radius is negative so the component doesn't render anything because of if (radius <= 0) { return; } at line 64.

If you don't precede me, I will make wrappers on Monday.

@ilariaventurini ilariaventurini mentioned this pull request May 11, 2020
8 tasks
ilariaventurini added a commit to accurat/carbon-charts that referenced this pull request May 11, 2020
* Radar setup

* Radar: create center and spokes

* Radar refactor

* Radar: Do not render anything if width or height are 0

* Radar: create blobs (wip)

* Radar: set colors

* Radar: refactor

* Radar: create x axes

* Radar: refactor

* Radar: fix color opacity

* Radar: Compute ticks with a minimum value

* Radar: draw y axis

* Radar: y axes start from the min tick value

* Radar: add exit for x axes

* Radar: blob creation refactor

* Radar: rename classes

* Radar: fix class name

* Radar: mini refactor

* Radar: fix blobs bind

* Radar: style changes

* Radar: fix min radius logic

* Radar: bind displayData to x axes not data

* Radar: add transitions

* Radar: add key labels (wip)

* Radar: change margin value

* Radar: fix labels position

* fix(core): fix backwards compatibility for pie/donut (carbon-design-system#576)

* fix(core): fix pie model to ignore dataset name when converting to tabular

* apply review changes

* provide all datapoint info to stacked bar tooltips (carbon-design-system#573)

* Merge pull request carbon-design-system#574 from theiliad/axis-formatters

* optimize axis tick formatting

* Update packages/core/src/components/axes/axis.ts

Co-Authored-By: natashadecoste <[email protected]>

* optimize axis formatter for time-scale when user has already provided one

Co-authored-by: natashadecoste <[email protected]>

* v0.30.12

* fix(core): Support for Horizontal Line and Scatter Chart

* fix line graph update issues

* v0.30.13

* Change method name

* v0.30.14

* Fix linting problem about methods order

* v0.30.15

* Radar: fix y axes transitions

* Code comments for tools.ts (carbon-design-system#416)

* update documentation for tools.ts

* docs(tools.ts): api documentation

Co-authored-by: Eliad Moosavi <[email protected]>

* remove bbox dependency on pie labels

* Merge pull request carbon-design-system#588 from accurat/invalid-time-value-bug

* Fix bug `RangeError Invalid time value`

* Extends domain refactor

* Time series: add example with addSpaceOnEdge = 0

* Rename functions

* Change method order

* v0.30.16

* v0.30.17

* Radar: use the new .join d3 method

* Use join to manage enter, update, exit logic

* Radar: fill missing data

* Radar: fix animations but need a refactor

* Radar: add events

* Radar: add tooltip

* Radar: miss an import

* Radar: use groupMapsTo

* Radar: number of spokes doesn't change

* Radar: use groupMapsTo for data mapping

* Radar: show NA when data are missing

* Radar: add y labels

* Radar: remove events on blobs

* Radar: left align

* Radar: create rectangles over x axis

* Radar: make x axis dashed on mouse over

* Radar: add dots

* Radar: move utility functions in another file

* Radar: fix radial label placement for negative angles

* Radar: refactor

* Radar: radialLabelPlacement refactor

* Radar: better logic to center chart

* Radar: fix x axis style when overed

* Radar: change blobs opacity

* Radar: create y axes transition

* Radar: create x axes transition

* Radar: create blobs transition

* Radar: x and y labels transition

* Radar: fix transitions

* Radar: more left space for y labels

* Radar: remove unused imports

* Radar: use the right transitions

* Radar: remove logs

* Radar: remove debug stuff

* Radar: update y labels

* Radar: fix Firefox bug

* Radar: move constants to configuration

* Radar: rename variables

* Radar: fix typo

* Radar: update comments

* Radar: move lodash methods inside Tools

* Radar: add examples and make radar more generic

* Radar: fix tooltip value

* Radar: fix tooltip data order

* Radar: fix radar default formatter

* Radar: fix showing dots

* Radar: use variable instead of recompute value

* Radar: create enums from textAnchor and dominantBaseline values

* Radar: put dots radius to radar configuration

* Radar: use tabs instead of spaces

* Radar: use helper function for accesing properties

* Radar: get property from options

Co-authored-by: natashadecoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
theiliad added a commit that referenced this pull request May 13, 2020
* Time series: add example to show the bug about labels

* Time series: fix bug about labels

* Revert "Time series: fix bug about labels"

This reverts commit 7c055fe.

* Fix linter problems

* Fix logic computing timeIntervalName

* Rename variables inside closestTimeIntervalName function

* Time series axis: use scale nice

* Remove useless demo example

* fix scatter interface

* v0.30.18

* Ruler interaction replacing Grid mouse listeners for comparative tooltips (#499)

* add ruler and axis tooltip

* move ruler logic to it's own graphFrameComponent

* refactor, add axis tooltip dynamic width, add matching elements hovering

* use getDomainValue instead of hardcoded accessor

* add snap to nearest point

* use getSVGElementSize to get backdrop height

* remove useless code

* add trigger mouseout in hideRuler to only highlight single hovered element

* WIP

* fix categorical scale elements mouseover trigger

* fix close points mouseover/out trigger

* fix import order and package version

* add more test data

* add crosshair cursor when ruler is active to better select data points

* refactor, more docs

* fix axis tooltip baseline

* minor refactor

Co-Authored-By: Zvonimir Fras <[email protected]>

* change ruler color

* a11y bubble

* only show ruler for continuous domain - wip

* add support for top axis

* no need to support discrete scales anymore

* make sure tooltip does not go out of axis bbox

* remove axis tooltip

* only show ruler if there's match with data points

* only show ruler on timescale linechart

* add filter for items getting null values at times, breaking tooltip

* refactor

* fix(core): update import path carbon-components

fix #546

* v0.30.7

* Merge pull request #529 from theiliad/tabular-data-format

* add basic tabular data format support

* string updates

* finalize line graph implementation with tabular data support

* cleanup

* finalize scatter graph with support for tabular data

* commented out ideas for the data format

* time-series support for tabular data within line graph

* scatter support

* optimize logic to find main vertical & horizontal axes

* remove primary & secondary labels

* bubble chart support

* grouped bar support

* horizontal grouped bar

* fix legend filtering

* fix bubble and scatter

* fix bar hover errors

* simple bar support

* simple bar time-series support

* horizontal simple bar support

* more graph support

* fix pie tooltip

* fix donut center value

* fix line & scatter demo ranges

* fix bar demos

* stacked data supprot

* backwards data format compatibility

* fix plotting of datums with a value of 0

* fixed stacked bar tooltip

* remove primary, secondary, useAsGraphDomain/Range

* Update packages/core/src/model.ts

Co-Authored-By: natashadecoste <[email protected]>

* apply suggested PR changes

* fix group bar positioning issues

* apply PR suggested changes

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* rename "identifier" to "mapsTo"

* add tutorials section to core storybook

* add links to errors

* remove unnecessary comment

* add custom color scale support

* apply PR suggested changes

* fix tests

Co-authored-by: natashadecoste <[email protected]>

* v0.30.8

* Apply suggestions from code review

Co-Authored-By: Eliad Moosavi <[email protected]>

* applied suggestions from review

* fix ruler css

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* Update packages/core/src/styles/components/_ruler.scss

Co-Authored-By: Eliad Moosavi <[email protected]>

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* add suggestions from review

* Update packages/core/demo/data/line.ts

Co-Authored-By: natashadecoste <[email protected]>

* Apply suggestions from code review

Co-Authored-By: natashadecoste <[email protected]>

* enable ruler on bubble and scatter

* disable mouse listeners on grid

* enable ruler for every scale type

* fix ruler on linear scales, refactor matches logic

Co-authored-by: Zvonimir Fras <[email protected]>
Co-authored-by: Natasha DeCoste <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>

* Ruler fix (#598)

* fix elements highlighting

* fix multiple matching points

* refactor

* dead code

* fix type

* refactor reducer logic

* more dead code

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* apply suggestion from review

* rename

* add orientation support

* fix wrong y offset in mouse coords

* better typing

Co-authored-by: Eliad Moosavi <[email protected]>

* fix vue type check for charting data

* Add Svelte wrappers (#569)

* feat: add svelte wrappers

* chore(svelte): format babelrc, rollup config

* chore(deps-dev): lock @storybook/svelte version

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(deps-dev): lock svelte version

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(deps-dev): lock svelte-loader version

Co-Authored-By: Eliad Moosavi <[email protected]>

* docs(svelte): fill in Codesandbox section

* chore(svelte): define maintainers, contributors in package.json

* docs(svelte): add codesandbox

* chore(svelte): update package version to 0.30.18

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(yarn): rebuild lockfile to fix errored ci

Co-authored-by: Eliad Moosavi <[email protected]>

* Update README.md

* v0.30.19

* Merge pull request #581 from accurat/radar

* Radar setup

* Radar: create center and spokes

* Radar refactor

* Radar: Do not render anything if width or height are 0

* Radar: create blobs (wip)

* Radar: set colors

* Radar: refactor

* Radar: create x axes

* Radar: refactor

* Radar: fix color opacity

* Radar: Compute ticks with a minimum value

* Radar: draw y axis

* Radar: y axes start from the min tick value

* Radar: add exit for x axes

* Radar: blob creation refactor

* Radar: rename classes

* Radar: fix class name

* Radar: mini refactor

* Radar: fix blobs bind

* Radar: style changes

* Radar: fix min radius logic

* Radar: bind displayData to x axes not data

* Radar: add transitions

* Radar: add key labels (wip)

* Radar: change margin value

* Radar: fix labels position

* fix(core): fix backwards compatibility for pie/donut (#576)

* fix(core): fix pie model to ignore dataset name when converting to tabular

* apply review changes

* provide all datapoint info to stacked bar tooltips (#573)

* Merge pull request #574 from theiliad/axis-formatters

* optimize axis tick formatting

* Update packages/core/src/components/axes/axis.ts

Co-Authored-By: natashadecoste <[email protected]>

* optimize axis formatter for time-scale when user has already provided one

Co-authored-by: natashadecoste <[email protected]>

* v0.30.12

* fix(core): Support for Horizontal Line and Scatter Chart

* fix line graph update issues

* v0.30.13

* Change method name

* v0.30.14

* Fix linting problem about methods order

* v0.30.15

* Radar: fix y axes transitions

* Code comments for tools.ts (#416)

* update documentation for tools.ts

* docs(tools.ts): api documentation

Co-authored-by: Eliad Moosavi <[email protected]>

* remove bbox dependency on pie labels

* Merge pull request #588 from accurat/invalid-time-value-bug

* Fix bug `RangeError Invalid time value`

* Extends domain refactor

* Time series: add example with addSpaceOnEdge = 0

* Rename functions

* Change method order

* v0.30.16

* v0.30.17

* Radar: use the new .join d3 method

* Use join to manage enter, update, exit logic

* Radar: fill missing data

* Radar: fix animations but need a refactor

* Radar: add events

* Radar: add tooltip

* Radar: miss an import

* Radar: use groupMapsTo

* Radar: number of spokes doesn't change

* Radar: use groupMapsTo for data mapping

* Radar: show NA when data are missing

* Radar: add y labels

* Radar: remove events on blobs

* Radar: left align

* Radar: create rectangles over x axis

* Radar: make x axis dashed on mouse over

* Radar: add dots

* Radar: move utility functions in another file

* Radar: fix radial label placement for negative angles

* Radar: refactor

* Radar: radialLabelPlacement refactor

* Radar: better logic to center chart

* Radar: fix x axis style when overed

* Radar: change blobs opacity

* Radar: create y axes transition

* Radar: create x axes transition

* Radar: create blobs transition

* Radar: x and y labels transition

* Radar: fix transitions

* Radar: more left space for y labels

* Radar: remove unused imports

* Radar: use the right transitions

* Radar: remove logs

* Radar: remove debug stuff

* Radar: update y labels

* Radar: fix Firefox bug

* Radar: move constants to configuration

* Radar: rename variables

* Radar: fix typo

* Radar: update comments

* Radar: move lodash methods inside Tools

* Radar: add examples and make radar more generic

* Radar: fix tooltip value

* Radar: fix tooltip data order

* Radar: fix radar default formatter

* Radar: fix showing dots

* Radar: use variable instead of recompute value

* Radar: create enums from textAnchor and dominantBaseline values

* Radar: put dots radius to radar configuration

* Radar: use tabs instead of spaces

* Radar: use helper function for accesing properties

* Radar: get property from options

Co-authored-by: natashadecoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>

* v0.30.20

Co-authored-by: Fei <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Luca Falasco <[email protected]>
Co-authored-by: Zvonimir Fras <[email protected]>
Co-authored-by: Natasha DeCoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: Eric Liu <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
theiliad added a commit to theiliad/carbon-charts that referenced this pull request May 17, 2021
* Radar setup

* Radar: create center and spokes

* Radar refactor

* Radar: Do not render anything if width or height are 0

* Radar: create blobs (wip)

* Radar: set colors

* Radar: refactor

* Radar: create x axes

* Radar: refactor

* Radar: fix color opacity

* Radar: Compute ticks with a minimum value

* Radar: draw y axis

* Radar: y axes start from the min tick value

* Radar: add exit for x axes

* Radar: blob creation refactor

* Radar: rename classes

* Radar: fix class name

* Radar: mini refactor

* Radar: fix blobs bind

* Radar: style changes

* Radar: fix min radius logic

* Radar: bind displayData to x axes not data

* Radar: add transitions

* Radar: add key labels (wip)

* Radar: change margin value

* Radar: fix labels position

* fix(core): fix backwards compatibility for pie/donut (carbon-design-system#576)

* fix(core): fix pie model to ignore dataset name when converting to tabular

* apply review changes

* provide all datapoint info to stacked bar tooltips (carbon-design-system#573)

* Merge pull request carbon-design-system#574 from theiliad/axis-formatters

* optimize axis tick formatting

* Update packages/core/src/components/axes/axis.ts

Co-Authored-By: natashadecoste <[email protected]>

* optimize axis formatter for time-scale when user has already provided one

Co-authored-by: natashadecoste <[email protected]>

* v0.30.12

* fix(core): Support for Horizontal Line and Scatter Chart

* fix line graph update issues

* v0.30.13

* Change method name

* v0.30.14

* Fix linting problem about methods order

* v0.30.15

* Radar: fix y axes transitions

* Code comments for tools.ts (carbon-design-system#416)

* update documentation for tools.ts

* docs(tools.ts): api documentation

Co-authored-by: Eliad Moosavi <[email protected]>

* remove bbox dependency on pie labels

* Merge pull request carbon-design-system#588 from accurat/invalid-time-value-bug

* Fix bug `RangeError Invalid time value`

* Extends domain refactor

* Time series: add example with addSpaceOnEdge = 0

* Rename functions

* Change method order

* v0.30.16

* v0.30.17

* Radar: use the new .join d3 method

* Use join to manage enter, update, exit logic

* Radar: fill missing data

* Radar: fix animations but need a refactor

* Radar: add events

* Radar: add tooltip

* Radar: miss an import

* Radar: use groupMapsTo

* Radar: number of spokes doesn't change

* Radar: use groupMapsTo for data mapping

* Radar: show NA when data are missing

* Radar: add y labels

* Radar: remove events on blobs

* Radar: left align

* Radar: create rectangles over x axis

* Radar: make x axis dashed on mouse over

* Radar: add dots

* Radar: move utility functions in another file

* Radar: fix radial label placement for negative angles

* Radar: refactor

* Radar: radialLabelPlacement refactor

* Radar: better logic to center chart

* Radar: fix x axis style when overed

* Radar: change blobs opacity

* Radar: create y axes transition

* Radar: create x axes transition

* Radar: create blobs transition

* Radar: x and y labels transition

* Radar: fix transitions

* Radar: more left space for y labels

* Radar: remove unused imports

* Radar: use the right transitions

* Radar: remove logs

* Radar: remove debug stuff

* Radar: update y labels

* Radar: fix Firefox bug

* Radar: move constants to configuration

* Radar: rename variables

* Radar: fix typo

* Radar: update comments

* Radar: move lodash methods inside Tools

* Radar: add examples and make radar more generic

* Radar: fix tooltip value

* Radar: fix tooltip data order

* Radar: fix radar default formatter

* Radar: fix showing dots

* Radar: use variable instead of recompute value

* Radar: create enums from textAnchor and dominantBaseline values

* Radar: put dots radius to radar configuration

* Radar: use tabs instead of spaces

* Radar: use helper function for accesing properties

* Radar: get property from options

Co-authored-by: natashadecoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
theiliad added a commit to theiliad/carbon-charts that referenced this pull request May 17, 2021
…labels-bug

* Time series: add example to show the bug about labels

* Time series: fix bug about labels

* Revert "Time series: fix bug about labels"

This reverts commit 7c055fe.

* Fix linter problems

* Fix logic computing timeIntervalName

* Rename variables inside closestTimeIntervalName function

* Time series axis: use scale nice

* Remove useless demo example

* fix scatter interface

* v0.30.18

* Ruler interaction replacing Grid mouse listeners for comparative tooltips (carbon-design-system#499)

* add ruler and axis tooltip

* move ruler logic to it's own graphFrameComponent

* refactor, add axis tooltip dynamic width, add matching elements hovering

* use getDomainValue instead of hardcoded accessor

* add snap to nearest point

* use getSVGElementSize to get backdrop height

* remove useless code

* add trigger mouseout in hideRuler to only highlight single hovered element

* WIP

* fix categorical scale elements mouseover trigger

* fix close points mouseover/out trigger

* fix import order and package version

* add more test data

* add crosshair cursor when ruler is active to better select data points

* refactor, more docs

* fix axis tooltip baseline

* minor refactor

Co-Authored-By: Zvonimir Fras <[email protected]>

* change ruler color

* a11y bubble

* only show ruler for continuous domain - wip

* add support for top axis

* no need to support discrete scales anymore

* make sure tooltip does not go out of axis bbox

* remove axis tooltip

* only show ruler if there's match with data points

* only show ruler on timescale linechart

* add filter for items getting null values at times, breaking tooltip

* refactor

* fix(core): update import path carbon-components

fix carbon-design-system#546

* v0.30.7

* Merge pull request carbon-design-system#529 from theiliad/tabular-data-format

* add basic tabular data format support

* string updates

* finalize line graph implementation with tabular data support

* cleanup

* finalize scatter graph with support for tabular data

* commented out ideas for the data format

* time-series support for tabular data within line graph

* scatter support

* optimize logic to find main vertical & horizontal axes

* remove primary & secondary labels

* bubble chart support

* grouped bar support

* horizontal grouped bar

* fix legend filtering

* fix bubble and scatter

* fix bar hover errors

* simple bar support

* simple bar time-series support

* horizontal simple bar support

* more graph support

* fix pie tooltip

* fix donut center value

* fix line & scatter demo ranges

* fix bar demos

* stacked data supprot

* backwards data format compatibility

* fix plotting of datums with a value of 0

* fixed stacked bar tooltip

* remove primary, secondary, useAsGraphDomain/Range

* Update packages/core/src/model.ts

Co-Authored-By: natashadecoste <[email protected]>

* apply suggested PR changes

* fix group bar positioning issues

* apply PR suggested changes

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* Update packages/core/demo/data/bar.ts

Co-Authored-By: natashadecoste <[email protected]>

* rename "identifier" to "mapsTo"

* add tutorials section to core storybook

* add links to errors

* remove unnecessary comment

* add custom color scale support

* apply PR suggested changes

* fix tests

Co-authored-by: natashadecoste <[email protected]>

* v0.30.8

* Apply suggestions from code review

Co-Authored-By: Eliad Moosavi <[email protected]>

* applied suggestions from review

* fix ruler css

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* Update packages/core/src/styles/components/_ruler.scss

Co-Authored-By: Eliad Moosavi <[email protected]>

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* add suggestions from review

* Update packages/core/demo/data/line.ts

Co-Authored-By: natashadecoste <[email protected]>

* Apply suggestions from code review

Co-Authored-By: natashadecoste <[email protected]>

* enable ruler on bubble and scatter

* disable mouse listeners on grid

* enable ruler for every scale type

* fix ruler on linear scales, refactor matches logic

Co-authored-by: Zvonimir Fras <[email protected]>
Co-authored-by: Natasha DeCoste <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>

* Ruler fix (carbon-design-system#598)

* fix elements highlighting

* fix multiple matching points

* refactor

* dead code

* fix type

* refactor reducer logic

* more dead code

* Update packages/core/src/components/axes/ruler.ts

Co-Authored-By: Eliad Moosavi <[email protected]>

* apply suggestion from review

* rename

* add orientation support

* fix wrong y offset in mouse coords

* better typing

Co-authored-by: Eliad Moosavi <[email protected]>

* fix vue type check for charting data

* Add Svelte wrappers (carbon-design-system#569)

* feat: add svelte wrappers

* chore(svelte): format babelrc, rollup config

* chore(deps-dev): lock @storybook/svelte version

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(deps-dev): lock svelte version

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(deps-dev): lock svelte-loader version

Co-Authored-By: Eliad Moosavi <[email protected]>

* docs(svelte): fill in Codesandbox section

* chore(svelte): define maintainers, contributors in package.json

* docs(svelte): add codesandbox

* chore(svelte): update package version to 0.30.18

Co-Authored-By: Eliad Moosavi <[email protected]>

* chore(yarn): rebuild lockfile to fix errored ci

Co-authored-by: Eliad Moosavi <[email protected]>

* Update README.md

* v0.30.19

* Merge pull request carbon-design-system#581 from accurat/radar

* Radar setup

* Radar: create center and spokes

* Radar refactor

* Radar: Do not render anything if width or height are 0

* Radar: create blobs (wip)

* Radar: set colors

* Radar: refactor

* Radar: create x axes

* Radar: refactor

* Radar: fix color opacity

* Radar: Compute ticks with a minimum value

* Radar: draw y axis

* Radar: y axes start from the min tick value

* Radar: add exit for x axes

* Radar: blob creation refactor

* Radar: rename classes

* Radar: fix class name

* Radar: mini refactor

* Radar: fix blobs bind

* Radar: style changes

* Radar: fix min radius logic

* Radar: bind displayData to x axes not data

* Radar: add transitions

* Radar: add key labels (wip)

* Radar: change margin value

* Radar: fix labels position

* fix(core): fix backwards compatibility for pie/donut (carbon-design-system#576)

* fix(core): fix pie model to ignore dataset name when converting to tabular

* apply review changes

* provide all datapoint info to stacked bar tooltips (carbon-design-system#573)

* Merge pull request carbon-design-system#574 from theiliad/axis-formatters

* optimize axis tick formatting

* Update packages/core/src/components/axes/axis.ts

Co-Authored-By: natashadecoste <[email protected]>

* optimize axis formatter for time-scale when user has already provided one

Co-authored-by: natashadecoste <[email protected]>

* v0.30.12

* fix(core): Support for Horizontal Line and Scatter Chart

* fix line graph update issues

* v0.30.13

* Change method name

* v0.30.14

* Fix linting problem about methods order

* v0.30.15

* Radar: fix y axes transitions

* Code comments for tools.ts (carbon-design-system#416)

* update documentation for tools.ts

* docs(tools.ts): api documentation

Co-authored-by: Eliad Moosavi <[email protected]>

* remove bbox dependency on pie labels

* Merge pull request carbon-design-system#588 from accurat/invalid-time-value-bug

* Fix bug `RangeError Invalid time value`

* Extends domain refactor

* Time series: add example with addSpaceOnEdge = 0

* Rename functions

* Change method order

* v0.30.16

* v0.30.17

* Radar: use the new .join d3 method

* Use join to manage enter, update, exit logic

* Radar: fill missing data

* Radar: fix animations but need a refactor

* Radar: add events

* Radar: add tooltip

* Radar: miss an import

* Radar: use groupMapsTo

* Radar: number of spokes doesn't change

* Radar: use groupMapsTo for data mapping

* Radar: show NA when data are missing

* Radar: add y labels

* Radar: remove events on blobs

* Radar: left align

* Radar: create rectangles over x axis

* Radar: make x axis dashed on mouse over

* Radar: add dots

* Radar: move utility functions in another file

* Radar: fix radial label placement for negative angles

* Radar: refactor

* Radar: radialLabelPlacement refactor

* Radar: better logic to center chart

* Radar: fix x axis style when overed

* Radar: change blobs opacity

* Radar: create y axes transition

* Radar: create x axes transition

* Radar: create blobs transition

* Radar: x and y labels transition

* Radar: fix transitions

* Radar: more left space for y labels

* Radar: remove unused imports

* Radar: use the right transitions

* Radar: remove logs

* Radar: remove debug stuff

* Radar: update y labels

* Radar: fix Firefox bug

* Radar: move constants to configuration

* Radar: rename variables

* Radar: fix typo

* Radar: update comments

* Radar: move lodash methods inside Tools

* Radar: add examples and make radar more generic

* Radar: fix tooltip value

* Radar: fix tooltip data order

* Radar: fix radar default formatter

* Radar: fix showing dots

* Radar: use variable instead of recompute value

* Radar: create enums from textAnchor and dominantBaseline values

* Radar: put dots radius to radar configuration

* Radar: use tabs instead of spaces

* Radar: use helper function for accesing properties

* Radar: get property from options

Co-authored-by: natashadecoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>

* v0.30.20

Co-authored-by: Fei <[email protected]>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: Luca Falasco <[email protected]>
Co-authored-by: Zvonimir Fras <[email protected]>
Co-authored-by: Natasha DeCoste <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: Eliad Moosavi <[email protected]>
Co-authored-by: Eric Liu <[email protected]>
Co-authored-by: Serena Girardi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Radar Chart Type Restyle: tree & radar
6 participants