fix(button-group): use direction independent border radius for children

telerik · Jan 10, 2022 · ca0f78d · ca0f78d
1 parent 02ea292
commit ca0f78d
Showing 1 changed file with 27 additions and 82 deletions.
diff --git a/packages/default/scss/button/_layout.scss b/packages/default/scss/button/_layout.scss
@@ -130,34 +130,33 @@
         -webkit-touch-callout: none;
         -webkit-tap-highlight-color: $rgba-transparent;
 
-        .k-button {
-            @include border-radius( 0 );
-        }
-        .k-button ~ .k-button {
+        > .k-button + .k-button {
             margin-inline-start: if( $button-border-width == 0, null, -$button-border-width );
         }
 
-        .k-button:hover,
-        .k-button.k-state-hover,
-        .k-button:active,
-        .k-button.k-state-active,
-        .k-button.k-state-selected,
-        .k-button:focus,
-        .k-button.k-state-focus {
+        > .k-button:hover,
+        > .k-button.k-state-hover,
+        > .k-button:focus,
+        > .k-button.k-state-focus,
+        > .k-button:active,
+        > .k-button.k-state-active,
+        > .k-button.k-state-selected {
             z-index: 2;
         }
 
-        .k-group-start,
-        .k-button:first-child {
-            @include border-left-radius( $button-border-radius );
+        .k-button:not(:first-child):not(:last-child) {
+            border-start-end-radius: 0;
+            border-end-end-radius: 0;
+            border-start-start-radius: 0;
+            border-end-start-radius: 0;
         }
-        .k-group-end,
-        .k-button:last-child {
-            @include border-right-radius( $button-border-radius );
+        > .k-button:first-child:not(:only-child) {
+            border-start-end-radius: 0;
+            border-end-end-radius: 0;
         }
-        .k-group-start.k-group-end,
-        .k-button:first-child:last-child {
-            @include border-radius( $button-border-radius );
+        > .k-button:last-child:not(:only-child) {
+            border-start-start-radius: 0;
+            border-end-start-radius: 0;
         }
 
     }
@@ -181,32 +180,13 @@
         flex-flow: row nowrap;
         vertical-align: middle;
 
-        .k-button {
-            @include border-radius( 0 );
-        }
-
-        // .k-button
-        > .k-button:first-child {
-            @include border-left-radius( $button-border-radius );
-        }
-        > .k-split-button-arrow,
-        > .k-button:last-child {
-            @include border-right-radius( $button-border-radius );
-            margin-inline-start: if( $button-border-width == 0, null, -$button-border-width );
+        .k-split-button-arrow {
             padding: $button-padding-y;
             width: auto;
             flex: none;
-        }
 
-        &[dir="rtl"] {
-            // k-button
-            > .k-button:first-child {
-                @include border-left-radius( 0 );
-                @include border-right-radius( $button-border-radius );
-            }
-            > .k-button:last-child {
-                @include border-right-radius( 0 );
-                @include border-left-radius( $button-border-radius );
+            .k-button-icon {
+                min-width: 0;
             }
         }
     }
@@ -282,46 +262,11 @@
     }
 
 
-    // RTL
-    .k-rtl {
-
-        // Button group
-        .k-button-group {
-
-            .k-button {
-                @include border-radius( 0 );
-            }
-
-            .k-group-start,
-            .k-button:first-child {
-                @include border-right-radius( $button-border-radius );
-            }
-            .k-group-end,
-            .k-button:last-child {
-                @include border-left-radius( $button-border-radius );
-            }
-            .k-group-start.k-group-end,
-            .k-button:first-child:last-child {
-                @include border-radius( $button-border-radius );
-            }
-
-        }
-
-        // Split button
-        .k-split-button {
-
-            .k-button {
-                @include border-radius( 0 );
-            }
-
-            // k-button
-            > .k-button:first-child {
-                @include border-right-radius( $button-border-radius );
-            }
-            > .k-split-button-arrow,
-            > .k-button:last-child {
-                @include border-left-radius( $button-border-radius );
-            }
+    // IE
+    .k-ie .k-button-group,
+    .k-ie .k-split-button {
+        .k-button {
+            @include border-radius( 0 );
         }
     }