Skip to content

Commit

Permalink
Merge pull request #20465 from DevitX/fix-18185
Browse files Browse the repository at this point in the history
fix(BarView): borderColor hides bar color with large data. close #18185
  • Loading branch information
Ovilia authored Nov 14, 2024
2 parents 18f8ea3 + 47e5928 commit 25d9570
Show file tree
Hide file tree
Showing 2 changed files with 122 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/chart/bar/BarView.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1176,7 +1176,8 @@ function createLarge(
el.barWidth = barWidth;
group.add(el);
el.useStyle(data.getVisual('style'));

// Stroke is rendered first to avoid overlapping with fill
el.style.stroke = null;
// Enable tooltip and user mouse/touch event handlers.
getECData(el).seriesIndex = seriesModel.seriesIndex;

Expand Down
120 changes: 120 additions & 0 deletions test/bar-large-border-fill.html

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

0 comments on commit 25d9570

Please sign in to comment.