Merge pull request #3193 from plotly/3189-vertical-modebar

fix size and alignment of vertical modebar to match horizontal modebar
plotly · Oct 31, 2018 · 6fda595 · 6fda595
2 parents 458f593 + bfe9cd3
commit 6fda595
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/build/plotcss.js b/build/plotcss.js
@@ -37,7 +37,8 @@ var rules = {
     "X .modebar-group": "float:left;display:inline-block;box-sizing:border-box;margin-left:8px;position:relative;vertical-align:middle;white-space:nowrap;",
     "X .modebar-btn": "position:relative;font-size:16px;padding:3px 4px;height:22px;cursor:pointer;line-height:normal;box-sizing:border-box;",
     "X .modebar-btn svg": "position:relative;top:2px;",
-    "X .modebar.vertical": "top:-1px;display:flex;flex-direction:column;flex-wrap:wrap;height:100%;align-content:flex-end;",
+    "X .modebar.vertical": "display:flex;flex-direction:column;flex-wrap:wrap;align-content:flex-end;max-height:100%;",
+    "X .modebar.vertical svg": "top:-1px;",
     "X .modebar.vertical .modebar-group": "display:block;float:none;margin-left:0px;margin-bottom:8px;",
     "X .modebar.vertical .modebar-group .modebar-btn": "display:block;text-align:center;",
     "X [data-title]:before,X [data-title]:after": "position:absolute;-webkit-transform:translate3d(0, 0, 0);-moz-transform:translate3d(0, 0, 0);-ms-transform:translate3d(0, 0, 0);-o-transform:translate3d(0, 0, 0);transform:translate3d(0, 0, 0);display:none;opacity:0;z-index:1001;pointer-events:none;top:110%;right:50%;",

diff --git a/src/css/_modebar.scss b/src/css/_modebar.scss
@@ -45,12 +45,14 @@
 }
 
 .modebar.vertical {
-    top: -1px;
     display: flex;
     flex-direction: column;
     flex-wrap: wrap;
-    height: 100%;
     align-content: flex-end;
+    max-height: 100%;
+    svg {
+      top: -1px;
+    }
     .modebar-group {
         display: block;
         float: none;