codejets
diff --git a/‎src/traces/bar/sieve.js
Copy file name to clipboardExpand all lines: src/traces/bar/sieve.js
+9-2Lines changed: 9 additions & 2 deletions b/‎src/traces/bar/sieve.js
Copy file name to clipboardExpand all lines: src/traces/bar/sieve.js
+9-2Lines changed: 9 additions & 2 deletions
diff --git a/‎src/traces/histogram/calc.js
Copy file name to clipboardExpand all lines: src/traces/histogram/calc.js
+7-1Lines changed: 7 additions & 1 deletion b/‎src/traces/histogram/calc.js
Copy file name to clipboardExpand all lines: src/traces/histogram/calc.js
+7-1Lines changed: 7 additions & 1 deletion
diff --git a/‎test/jasmine/tests/histogram_test.js
Copy file name to clipboardExpand all lines: test/jasmine/tests/histogram_test.js
+24Lines changed: 24 additions & 0 deletions b/‎test/jasmine/tests/histogram_test.js
Copy file name to clipboardExpand all lines: test/jasmine/tests/histogram_test.js
+24Lines changed: 24 additions & 0 deletions
@@ -30,19 +30,26 @@ function Sieve(traces, separateNegativeValues, dontMergeOverlappingData) {
     this.separateNegativeValues = separateNegativeValues;
     this.dontMergeOverlappingData = dontMergeOverlappingData;
 
+    // for single-bin histograms - see histogram/calc
+    var width1 = Infinity;
+
     var positions = [];
     for(var i = 0; i < traces.length; i++) {
         var trace = traces[i];
         for(var j = 0; j < trace.length; j++) {
             var bar = trace[j];
             if(bar.p !== BADNUM) positions.push(bar.p);
         }
+        if(trace[0] && trace[0].width1) {
+            width1 = Math.min(trace[0].width1, width1);
+        }
     }
     this.positions = positions;
 
-    var dv = Lib.distinctVals(this.positions);
+    var dv = Lib.distinctVals(positions);
     this.distinctPositions = dv.vals;
-    this.minDiff = dv.minDiff;
+    if(dv.vals.length === 1 && width1) this.minDiff = width1;
+    else this.minDiff = Math.min(dv.minDiff, width1);
 
     this.binWidth = this.minDiff;
 
 
@@ -135,7 +135,7 @@ module.exports = function calc(gd, trace) {
             break;
         }
     }
-    for(i = seriesLen - 1; i > firstNonzero; i--) {
+    for(i = seriesLen - 1; i >= firstNonzero; i--) {
         if(size[i]) {
             lastNonzero = i;
             break;
@@ -149,6 +149,12 @@ module.exports = function calc(gd, trace) {
         }
     }
 
+    if(cd.length === 1) {
+        // when we collapse to a single bin, calcdata no longer describes bin size
+        // so we need to explicitly specify it
+        cd[0].width1 = Axes.tickIncrement(cd[0].p, binSpec.size, false, calendar) - cd[0].p;
+    }
+
     arraysToCalcdata(cd, trace);
 
     return cd;
 
@@ -269,6 +269,17 @@ describe('Test histogram', function() {
             expect(out.length).toEqual(9001);
         });
 
+        it('handles single-bin data without extra bins', function() {
+            var out = _calc({
+                x: [2.1, 3, 3.9],
+                xbins: {start: 0, end: 10, size: 2}
+            });
+
+            expect(out).toEqual([
+                {b: 0, p: 3, s: 3, width1: 2}
+            ]);
+        });
+
         function calcPositions(opts, extraTraces) {
             return _calc(opts, extraTraces).map(function(v) { return v.p; });
         }
@@ -554,5 +565,18 @@ describe('Test histogram', function() {
             .catch(fail)
             .then(done);
         });
+
+        it('give the right bar width for single-bin histograms', function(done) {
+            Plotly.newPlot(gd, [{
+                type: 'histogram',
+                x: [3, 3, 3],
+                xbins: {start: 0, end: 10, size: 2}
+            }])
+            .then(function() {
+                expect(gd._fullLayout.xaxis.range).toBeCloseToArray([2, 4], 3);
+            })
+            .catch(fail)
+            .then(done);
+        });
     });
 });
Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ module.exports = function calc(gd, trace) {`
`135`	`135`	`break;`
`136`	`136`	`}`
`137`	`137`	`}`
`138`		`- for(i = seriesLen - 1; i > firstNonzero; i--) {`
	`138`	`+ for(i = seriesLen - 1; i >= firstNonzero; i--) {`
`139`	`139`	`if(size[i]) {`
`140`	`140`	`lastNonzero = i;`
`141`	`141`	`break;`
`@@ -149,6 +149,12 @@ module.exports = function calc(gd, trace) {`
`149`	`149`	`}`
`150`	`150`	`}`
`151`	`151`
	`152`	`+ if(cd.length === 1) {`
	`153`	`+ // when we collapse to a single bin, calcdata no longer describes bin size`
	`154`	`+ // so we need to explicitly specify it`
	`155`	`+ cd[0].width1 = Axes.tickIncrement(cd[0].p, binSpec.size, false, calendar) - cd[0].p;`
	`156`	`+ }`
	`157`	`+`
`152`	`158`	`arraysToCalcdata(cd, trace);`
`153`	`159`
`154`	`160`	`return cd;`