Skip to content

Commit

Permalink
Would like to make it clear that options can be reactive, just n… (ap…
Browse files Browse the repository at this point in the history
…ertureless#558)

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.