Skip to content

Commit

Permalink
Merge pull request #12850 from plainheart/fix-lines-mergeOption-12836
Browse files Browse the repository at this point in the history
fix(lines): lines series shouldn't disappear after `setOption({})`. (#12836)
  • Loading branch information
pissang authored Aug 4, 2020
2 parents 57914fb + 42ba17f commit 419336e
Show file tree
Hide file tree
Showing 2 changed files with 104 additions and 3 deletions.
4 changes: 1 addition & 3 deletions src/chart/lines/LinesSeries.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,6 @@ var LinesSeries = SeriesModel.extend({
},

mergeOption: function (option) {
// The input data may be null/undefined.
option.data = option.data || [];

compatEc2(option);

Expand Down Expand Up @@ -333,4 +331,4 @@ var LinesSeries = SeriesModel.extend({
}
});

export default LinesSeries;
export default LinesSeries;
103 changes: 103 additions & 0 deletions test/lines-mergeOption.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 419336e

Please sign in to comment.