Bar chart with second y axis overlaps data series (#4150)

getredash · Aug 5, 2020 · eb603f6 · eb603f6
1 parent 6c00f7c
commit eb603f6
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/default.json b/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/default.json
@@ -38,6 +38,7 @@
     "series": [
       {
         "visible": true,
+        "offsetgroup": "0",
         "type": "bar",
         "name": "a",
         "x": ["x1", "x2", "x3", "x4"],

diff --git a/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/normalized.json b/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/normalized.json
@@ -48,6 +48,7 @@
     "series": [
       {
         "visible": true,
+        "offsetgroup": "0",
         "type": "bar",
         "name": "a",
         "x": ["x1", "x2", "x3", "x4"],
@@ -63,6 +64,7 @@
       },
       {
         "visible": true,
+        "offsetgroup": "1",
         "type": "bar",
         "name": "b",
         "x": ["x1", "x2", "x3", "x4"],

diff --git a/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/stacked.json b/viz-lib/src/visualizations/chart/plotly/fixtures/prepareData/bar/stacked.json
@@ -48,6 +48,7 @@
     "series": [
       {
         "visible": true,
+        "offsetgroup": "0",
         "type": "bar",
         "name": "a",
         "x": ["x1", "x2", "x3", "x4"],
@@ -63,6 +64,7 @@
       },
       {
         "visible": true,
+        "offsetgroup": "1",
         "type": "bar",
         "name": "b",
         "x": ["x1", "x2", "x3", "x4"],

diff --git a/viz-lib/src/visualizations/chart/plotly/prepareDefaultData.js b/viz-lib/src/visualizations/chart/plotly/prepareDefaultData.js
@@ -1,4 +1,4 @@
-import { isNil, extend, each, includes, map, sortBy } from "lodash";
+import { isNil, extend, each, includes, map, sortBy, toString } from "lodash";
 import chooseTextColorForBackground from "@/lib/chooseTextColorForBackground";
 import { ColorPaletteArray } from "@/visualizations/ColorPalette";
 import { cleanNumber, normalizeValue, getSeriesAxis } from "./utils";
@@ -16,8 +16,9 @@ function getHoverInfoPattern(options) {
   return result;
 }
 
-function prepareBarSeries(series, options) {
+function prepareBarSeries(series, options, additionalOptions) {
   series.type = "bar";
+  series.offsetgroup = toString(additionalOptions.index);
   if (options.showDataLabels) {
     series.textposition = "inside";
   }