display mid in the middle

Author: archmoj

src/traces/box/hover.js

@@ -145,10 +145,10 @@ function hoverOnBoxes(pointData, xval, yval, hovermode) {
 
     // labels with euqual values (e.g. when min === q1) should be presented in certain order
     var attrs =
-        (hasFences && hasMean) ? ['max', 'uf', 'q3', 'mean', 'q1', 'lf', 'min'] :
-        (hasFences && !hasMean) ? ['max', 'uf', 'q3', 'q1', 'lf', 'min'] :
-        (!hasFences && hasMean) ? ['max', 'q3', 'mean', 'q1', 'min'] :
-        ['max', 'q3', 'q1', 'min'];
+        (hasFences && hasMean) ? ['max', 'uf', 'q3', 'med', 'mean', 'q1', 'lf', 'min'] :
+        (hasFences && !hasMean) ? ['max', 'uf', 'q3', 'med', 'q1', 'lf', 'min'] :
+        (!hasFences && hasMean) ? ['max', 'q3', 'med', 'mean', 'q1', 'min'] :
+        ['max', 'q3', 'med', 'q1', 'min'];
 
     var rev = vAxis.range[1] < vAxis.range[0];
 
@@ -159,9 +159,6 @@ function hoverOnBoxes(pointData, xval, yval, hovermode) {
         attrs.reverse();
     }
 
-    // always put median at the start so that it get the extra label
-    attrs = ['med'].concat(attrs);
-
     var closeBoxData = [];
     for(var i = 0; i < attrs.length; i++) {
         var attr = attrs[i];
@@ -187,9 +184,15 @@ function hoverOnBoxes(pointData, xval, yval, hovermode) {
         }
 
         // only keep name and spikes on the first item (median)
-        pointData.name = '';
-        pointData.spikeDistance = undefined;
-        pointData[spikePosAttr] = undefined;
+        if(attr === 'med') {
+            pointData.name = '';
+            pointData.spikeDistance = undefined;
+            pointData[spikePosAttr] = undefined;
+        } else {
+            pointData2.name = '';
+            pointData2.spikeDistance = undefined;
+            pointData2[spikePosAttr] = undefined;
+        }
 
         // no hovertemplate support yet
         pointData2.hovertemplate = false;

test/jasmine/tests/box_test.js

@@ -782,10 +782,11 @@ describe('Test box hover:', function() {
         },
         pos: [350, 200],
         nums: [
+            '23.25',
             'median: 8.15', 'min: 0.75', 'q1: 6.8',
             'q3: 10.25', 'max: 23.25', 'lower fence: 5.25', 'upper fence: 12'
         ],
-        name: ['', '', '', '', '', '', ''],
+        name: ['', '', '', '', '', '', '', ''],
         axis: 'trace 0'
     }, {
         desc: 'with overlaid boxes',

test/jasmine/tests/hover_label_test.js

@@ -4680,10 +4680,10 @@ describe('hovermode: (x|y)unified', function() {
                     'min: 1',
                     'lower fence: 1',
                     'q1: 1',
+                    'trace 1 : median: 1',
                     'q3: 1',
                     'upper fence: 1',
                     'max: 1',
-                    'trace 1 : median: 1',
                     'trace 3 : 2',
                     'trace 2 : 2',
                     'trace 5 : 2',

test/jasmine/tests/violin_test.js

@@ -552,10 +552,10 @@ describe('Test violin hover:', function() {
         name: ['', '', '', '', '', '', ''],
         axis: 'Sat',
         hoverLabelPos: [
-            [363, 270],
             [338, 270],
             [377, 270],
             [351, 270],
+            [363, 270],
             [345, 270],
             [385, 270],
             [347, 270]