Skip to content

Commit

Permalink
Would like to make it clear that options can be reactive, just n… (#558)
Browse files Browse the repository at this point in the history
Would like to make it clear that options can be reactive, just not through the mixin.
  • Loading branch information
apertureless authored Oct 8, 2019
2 parents 2febeb8 + d964996 commit 6981876
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/guide/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ The reactivity system at its current state is not **robust**. You will run into

#### Options

The `options` object is not reactive right now. So if you dynamically change the chart options, they will not be recognized by the mixin.
The `options` object is not reactive right now. So if you dynamically change the chart options, they will not be recognized by the mixin. You can create a watcher to destroy and rerender when chart options are updated.

If you are using the mixin you need to pass in your options as a prop names `options`. This is important because the mixin will call chart.js `update()` method or destroy and render a new chart. If the mixin renders a new chart it calls `this.renderChart(this.chartData, this.options)`.

Expand Down

0 comments on commit 6981876

Please sign in to comment.