From 54f317f17f6e840aad29a03b0adcb8066ae7040b Mon Sep 17 00:00:00 2001 From: Joelius300 Date: Sun, 4 Oct 2020 22:04:39 +0200 Subject: [PATCH] Remove unnecessary divs around canvas 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 --- src/ChartJs.Blazor/Charts/Chart.razor | 6 +----- src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css | 8 -------- 2 files changed, 1 insertion(+), 13 deletions(-) delete mode 100644 src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css diff --git a/src/ChartJs.Blazor/Charts/Chart.razor b/src/ChartJs.Blazor/Charts/Chart.razor index 367e6637..92a51dcc 100644 --- a/src/ChartJs.Blazor/Charts/Chart.razor +++ b/src/ChartJs.Blazor/Charts/Chart.razor @@ -1,7 +1,3 @@ @typeparam TConfig -
-
- -
-
+ diff --git a/src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css b/src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css deleted file mode 100644 index b72900bb..00000000 --- a/src/ChartJs.Blazor/wwwroot/ChartJSBlazor.css +++ /dev/null @@ -1,8 +0,0 @@ -.chart-container { - /* Always take up the full space. The developer has to restrict space themself because it's impossible - to know how much space the chart should take up in the website this library is used. */ - position: relative; - margin: auto; - height: 100%; - width: 100%; -} \ No newline at end of file