From 0c6da96f0ce5ddbbfa58d9efed96ac62f2a22221 Mon Sep 17 00:00:00 2001 From: Robbie Cronin Date: Wed, 1 May 2019 09:13:53 +0800 Subject: Fix namespace issue Rename 2 conflicting Options variables in Charts and Tabulator packages to ChartOptions and TabulatorOptions respectively --- .../src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'kvision-modules/kvision-chart/src') diff --git a/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt b/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt index 0925a0c8..a598bc5d 100644 --- a/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt +++ b/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt @@ -728,7 +728,7 @@ fun ChartScales.toJs(i18nTranslator: (String) -> (String)): dynamic { /** * Chart options. */ -data class Options( +data class ChartOptions( val responsive: Boolean = true, val responsiveAnimationDuration: Int = 0, val aspectRatio: Int = 2, -- cgit From 37e980522d2512b25e0bfa074fce42732d5c0596 Mon Sep 17 00:00:00 2001 From: Robbie Cronin Date: Wed, 1 May 2019 14:13:09 +0800 Subject: Update missed namespace changes Change Options to ChartOptions to match new variable name --- .../src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'kvision-modules/kvision-chart/src') diff --git a/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt b/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt index a598bc5d..df3b7973 100644 --- a/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt +++ b/kvision-modules/kvision-chart/src/main/kotlin/pl/treksoft/kvision/chart/Configuration.kt @@ -758,7 +758,7 @@ data class ChartOptions( * An extension function to convert configuration class to JS object. */ @Suppress("ComplexMethod") -fun Options.toJs(i18nTranslator: (String) -> (String)): dynamic { +fun ChartOptions.toJs(i18nTranslator: (String) -> (String)): dynamic { return obj { this.responsive = responsive this.responsiveAnimationDuration = responsiveAnimationDuration @@ -889,7 +889,7 @@ data class Configuration( val type: ChartType, val dataSets: List, val labels: List? = null, - val options: Options? = null + val options: ChartOptions? = null ) /** -- cgit From 63767fec9660bdcbfd041ec28fb76ba95b322f2a Mon Sep 17 00:00:00 2001 From: Robbie Cronin Date: Wed, 1 May 2019 17:42:04 +0800 Subject: Update ChartCanvasSpec and ChartSpec for new ChartOptions name --- .../src/test/kotlin/test/pl/treksoft/kvision/chart/ChartCanvasSpec.kt | 4 ++-- .../src/test/kotlin/test/pl/treksoft/kvision/chart/ChartSpec.kt | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'kvision-modules/kvision-chart/src') diff --git a/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartCanvasSpec.kt b/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartCanvasSpec.kt index 9ea2ee6e..6fae54ad 100644 --- a/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartCanvasSpec.kt +++ b/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartCanvasSpec.kt @@ -25,7 +25,7 @@ import pl.treksoft.kvision.chart.ChartCanvas import pl.treksoft.kvision.chart.ChartType import pl.treksoft.kvision.chart.Configuration import pl.treksoft.kvision.chart.DataSets -import pl.treksoft.kvision.chart.Options +import pl.treksoft.kvision.chart.ChartOptions import pl.treksoft.kvision.panel.Root import test.pl.treksoft.kvision.DomSpec import kotlin.browser.document @@ -62,7 +62,7 @@ class ChartCanvasSpec : DomSpec { configuration = Configuration( ChartType.SCATTER, listOf(DataSets(label = "Chart", data = listOf(0, 1))), - options = Options(responsive = false) + options = ChartOptions(responsive = false) ) ) root.add(chart) diff --git a/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartSpec.kt b/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartSpec.kt index 2e508107..7974ea78 100644 --- a/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartSpec.kt +++ b/kvision-modules/kvision-chart/src/test/kotlin/test/pl/treksoft/kvision/chart/ChartSpec.kt @@ -25,7 +25,7 @@ import pl.treksoft.kvision.chart.Chart import pl.treksoft.kvision.chart.ChartType import pl.treksoft.kvision.chart.Configuration import pl.treksoft.kvision.chart.DataSets -import pl.treksoft.kvision.chart.Options +import pl.treksoft.kvision.chart.ChartOptions import pl.treksoft.kvision.panel.Root import pl.treksoft.kvision.utils.px import test.pl.treksoft.kvision.DomSpec @@ -65,7 +65,7 @@ class ChartSpec : DomSpec { Configuration( ChartType.SCATTER, listOf(DataSets(label = "Chart", data = listOf(0, 1))), - options = Options(responsive = false) + options = ChartOptions(responsive = false) ), 300, 600 ) root.add(chart) -- cgit