add comment regarding not showing zero span bars

archmoj · archmoj · commit c1ee8c227905 · 2020-02-11T12:25:28.000-05:00
diff --git a/src/traces/bar/plot.js b/src/traces/bar/plot.js
@@ -191,7 +191,13 @@ function plot(gd, plotinfo, cdModule, traceLayer, opts, makeOnCompleteCallback)
             }
 
             function expandToVisible(v, vc, hideZeroSpan) {
-                if(hideZeroSpan && v === vc) return v;
+                if(hideZeroSpan && v === vc) {
+                    // should not expand zero span bars
+                    // when start and end positions are identical
+                    // i.e. for vertical when y0 === y1
+                    // and for horizontal when x0 === x1
+                    return v;
+                }
 
                 // if it's not in danger of disappearing entirely,
                 // round more precisely
