Do not override outside when base is set

When bars are in stack or relative mode, 'outside' position
should not be overriden when a base is set.

Author: eivindjahren

src/traces/bar/plot.js

@@ -199,7 +199,7 @@ function appendBarText(gd, bar, calcTrace, i, x0, x1, y0, y1) {
         textHeight;
 
     if(textPosition === 'outside') {
-        if(!isOutmostBar) textPosition = 'inside';
+        if(!isOutmostBar && !calcBar.hasB) textPosition = 'inside';
     }
 
     if(textPosition === 'auto') {

test/jasmine/tests/bar_test.js

@@ -901,6 +901,68 @@ describe('A bar plot', function() {
         .then(done);
     });
 
+    it('Pushes outside text relative bars inside when not outmost', function(done) {
+        var data = [{
+            x: [1, 1, 2, 2],
+            y: [10, 20, 20, 10],
+            type: 'bar',
+            text: ['a', 'b', 'c', 'd'],
+            textposition: ['outside', 'inside', 'outside', 'inside'],
+        }];
+        var layout = {barmode: 'relative'};
+
+        Plotly.plot(gd, data, layout).then(function() {
+            var traceNodes = getAllTraceNodes(gd),
+                barNodes = getAllBarNodes(traceNodes[0]),
+                foundTextNodes;
+
+            for(var i = 0; i < barNodes.length; i++) {
+                var barNode = barNodes[i],
+                    pathNode = barNode.querySelector('path'),
+                    textNode = barNode.querySelector('text');
+                if(textNode) {
+                    foundTextNodes = true;
+                    assertTextIsInsidePath(textNode, pathNode);
+                }
+            }
+
+            expect(foundTextNodes).toBe(true);
+        })
+        .catch(failTest)
+        .then(done);
+    });
+
+    it('does not push text inside when base is set', function(done) {
+        var data = [{
+            x: [1, 1, 2, 2],
+            y: [10, 20, 20, 10],
+            base: [1, 2, 3, 4],
+            type: 'bar',
+            text: ['a', 'b', 'c', 'd'],
+            textposition: ['outside', 'outside', 'outside', 'outside'],
+        }];
+        var layout = {barmode: 'relative'};
+
+        Plotly.plot(gd, data, layout).then(function() {
+            var traceNodes = getAllTraceNodes(gd),
+                barNodes = getAllBarNodes(traceNodes[0]),
+                foundTextNodes;
+
+            for(var i = 0; i < barNodes.length; i++) {
+                var barNode = barNodes[i],
+                    pathNode = barNode.querySelector('path'),
+                    textNode = barNode.querySelector('text');
+                if(textNode) {
+                    foundTextNodes = true;
+                    assertTextIsAbovePath(textNode, pathNode);
+                }
+            }
+
+            expect(foundTextNodes).toBe(true);
+        })
+        .catch(failTest)
+        .then(done);
+    });
     it('should show bar texts (outside case)', function(done) {
         var data = [{
             y: [10, -20, 30],