[core] fix(ControlGroup): margin around dividers (#4215)

palantir · Jul 22, 2020 · 8a8de97 · 8a8de97 · blueprint-bot · Jul 22, 2020
1 parent c9a6e68
commit 8a8de97
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/packages/core/src/components/forms/_control-group.scss b/packages/core/src/components/forms/_control-group.scss
@@ -4,6 +4,7 @@
 @import "../../common/variables";
 @import "../button/common";
 @import "./common";
+@import "../divider/divider";
 
 /*
 Control groups
@@ -164,15 +165,20 @@ Styleguide control-group
 
   // have consecutive elements share a border
   &:not(.#{$ns}-vertical) {
-    > * {
+    > *:not(.#{$ns}-divider) {
       margin-right: -$button-border-width;
     }
 
+    > .#{$ns}-divider:not(:first-child) {
+      margin-left: $divider-margin + $button-border-width;
+    }
+
     .#{$ns}-dark & {
-      > * {
+      > *:not(.#{$ns}-divider) {
         margin-right: 0;
       }
 
+
       // consecutive buttons within a button group look okay out of the box, but
       // we need need to make a small correction for non-grouped buttons. this
       // replicates what's already done in dark theme button groups.