Skip to content

Commit

Permalink
Remove unnecessary divs around canvas
Browse files Browse the repository at this point in the history
These can break a lot of stuff especially since one has a class named "row" which is definitely used in some frameworks. As far as I understand it also bringts no actual benefits.
The issue with responsiveness (#128) is still very much present so this will change again (hopefully before the next release). I guess it's a breaking change since some people have been using the row class now.. aren't we glad 2.0 is up next
  • Loading branch information
Joelius300 committed Oct 4, 2020
1 parent 03d664a commit 54f317f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 13 deletions.
6 changes: 1 addition & 5 deletions src/ChartJs.Blazor/Charts/Chart.razor
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
@typeparam TConfig

<div class="row">
<div class="chart-container">
<canvas id="@Config.CanvasId" width="@Width" height="@Height"></canvas>
</div>
</div>
<canvas id="@Config.CanvasId" width="@Width" height="@Height"></canvas>
8 changes: 0 additions & 8 deletions src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css

This file was deleted.

0 comments on commit 54f317f

Please sign in to comment.