Revert "BugFix: Use first-of-type for buttongroup selector (#5343)"

This reverts commit 70005b8.
primer · Dec 10, 2024 · 3be386c · 3be386c
1 parent 8b4f03c
commit 3be386c
Show file tree

Hide file tree

Showing 13 changed files with 6 additions and 6 deletions.
diff --git a/...erlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-colorblind-linux.png b/...erlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-colorblind-linux.png
diff --git a/...s/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-dimmed-linux.png b/...s/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-dimmed-linux.png
diff --git a/...ay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-high-contrast-linux.png b/...ay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-high-contrast-linux.png
diff --git a/...mponents/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-linux.png b/...mponents/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-linux.png
diff --git a/...erlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-tritanopia-linux.png b/...erlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-dark-tritanopia-linux.png
diff --git a/...rlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-colorblind-linux.png b/...rlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-colorblind-linux.png
diff --git a/...y.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-high-contrast-linux.png b/...y.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-high-contrast-linux.png
diff --git a/...ponents/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-linux.png b/...ponents/Overlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-linux.png
diff --git a/...rlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-tritanopia-linux.png b/...rlay.test.ts-snapshots/Overlay-Memex-Nested-Overlays-light-tritanopia-linux.png
diff --git a/.../Overlay.test.ts-snapshots/Overlay-Nested-Overlays-dark-high-contrast-linux.png b/.../Overlay.test.ts-snapshots/Overlay-Nested-Overlays-dark-high-contrast-linux.png
diff --git a/...Overlay.test.ts-snapshots/Overlay-Nested-Overlays-light-high-contrast-linux.png b/...Overlay.test.ts-snapshots/Overlay-Nested-Overlays-light-high-contrast-linux.png
diff --git a/packages/react/src/ButtonGroup/ButtonGroup.module.css b/packages/react/src/ButtonGroup/ButtonGroup.module.css
@@ -3,18 +3,18 @@
   vertical-align: middle;
   isolation: isolate;
 
-  & > *:not([data-loading-wrapper]):is(button, a) {
+  & > *:not([data-loading-wrapper]) {
     /* stylelint-disable-next-line primer/spacing */
     margin-inline-end: -1px;
     position: relative;
     border-radius: 0;
 
-    &:first-of-type {
+    &:first-child {
       border-top-left-radius: var(--borderRadius-medium);
       border-bottom-left-radius: var(--borderRadius-medium);
     }
 
-    &:last-of-type {
+    &:last-child {
       border-top-right-radius: var(--borderRadius-medium);
       border-bottom-right-radius: var(--borderRadius-medium);
     }

diff --git a/packages/react/src/ButtonGroup/ButtonGroup.tsx b/packages/react/src/ButtonGroup/ButtonGroup.tsx
@@ -19,17 +19,17 @@ const StyledButtonGroup = toggleStyledComponent(
     vertical-align: middle;
     isolation: isolate;
 
-    && > *:not([data-loading-wrapper]):is(button, a) {
+    && > *:not([data-loading-wrapper]) {
       margin-inline-end: -1px;
       position: relative;
       border-radius: 0;
 
-      :first-of-type {
+      :first-child {
         border-top-left-radius: ${get('radii.2')};
         border-bottom-left-radius: ${get('radii.2')};
       }
 
-      :last-of-type {
+      :last-child {
         border-top-right-radius: ${get('radii.2')};
         border-bottom-right-radius: ${get('radii.2')};
       }