BUG: Also apply barplot unit to border lens

WHOOPS
biocore · Nov 10, 2020 · 56552f5 · 56552f5
1 parent 8369083
commit 56552f5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/empress/support_files/js/barplot-panel-handler.js b/empress/support_files/js/barplot-panel-handler.js
@@ -140,8 +140,8 @@ define([
         this.borderColor = Colorer.rgbToFloat([255, 255, 255]);
 
         // ... and to having a length of whatever the default barplot layer
-        // length divided by 2 is :)
-        this.borderLength = BarplotLayer.DEFAULT_LENGTH / 2;
+        // length divided by 10 is :)
+        this.borderLength = BarplotLayer.DEFAULT_LENGTH / 10;
 
         // Now, initialize the border options UI accordingly
         this.initBorderOptions();

diff --git a/empress/support_files/js/empress.js b/empress/support_files/js/empress.js
@@ -1982,7 +1982,7 @@ define([
     ) {
         var borderColor = this._barplotPanel.borderColor;
         var borderLength = this._barplotPanel.borderLength;
-        var maxD = prevLayerMaxD + borderLength;
+        var maxD = prevLayerMaxD + borderLength * this._barplotUnit;
         // TODO: Should be changed when the ability to change the background
         // color is added. Basically, we get a "freebie" if the border color
         // matches the background color, and we don't need to draw anything --