revert xaxis.tickPlacement to 'between' by default - fixes #305

junedchhipa · junedchhipa · commit 1c8f73438103 · 2019-02-10T16:17:11.000+05:30
diff --git a/src/modules/settings/Config.js b/src/modules/settings/Config.js
@@ -102,7 +102,7 @@ export default class Config {
           opts.chart.type === 'scatter') &&
         !combo.comboChartsHasBars &&
         opts.xaxis.type !== 'datetime' &&
-        opts.xaxis.tickPlacement !== 'between'
+        opts.xaxis.tickPlacement === 'on'
       ) {
         defaults.convertCatToNumeric()
       }
diff --git a/src/modules/settings/Options.js b/src/modules/settings/Options.js
@@ -777,7 +777,7 @@ export default class Options {
           offsetY: 0
         },
         tickAmount: undefined,
-        tickPlacement: 'on',
+        tickPlacement: 'between',
         min: undefined,
         max: undefined,
         range: undefined,

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ export default class Config {`
`102`	`102`	`opts.chart.type === 'scatter') &&`
`103`	`103`	`!combo.comboChartsHasBars &&`
`104`	`104`	`opts.xaxis.type !== 'datetime' &&`
`105`		`- opts.xaxis.tickPlacement !== 'between'`
	`105`	`+ opts.xaxis.tickPlacement === 'on'`
`106`	`106`	`) {`
`107`	`107`	`defaults.convertCatToNumeric()`
`108`	`108`	`}`