Drop “display: none” from details element contents styling (#2624)

sideshowbarker · web-flow · commit 7e6253292c3d · 2024-07-09T10:11:01.000+01:00
Fixes #2592
diff --git a/.changeset/slow-rabbits-hunt.md b/.changeset/slow-rabbits-hunt.md
@@ -0,0 +1,5 @@
+---
+"@primer/css": patch
+---
+
+drop “display: none” from details element contents styling
diff --git a/src/base/base.scss b/src/base/base.scss
@@ -96,13 +96,6 @@ details {
   summary {
     cursor: pointer;
   }
-
-  &:not([open]) {
-    // Set details content hidden by default for browsers that don't do this
-    > *:not(summary) {
-      display: none;
-    }
-  }
 }
 
 // global focus styles

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@primer/css": patch
 +---
++
 +drop “display: none” from details element contents styling
Original file line number	Diff line number	Diff line change
`@@ -96,13 +96,6 @@ details {`
`96`	`96`	`summary {`
`97`	`97`	`cursor: pointer;`
`98`	`98`	`}`
`99`		`-`
`100`		`- &:not([open]) {`
`101`		`- // Set details content hidden by default for browsers that don't do this`
`102`		`- > *:not(summary) {`
`103`		`- display: none;`
`104`		`- }`
`105`		`- }`
`106`	`99`	`}`
`107`	`100`
`108`	`101`	`// global focus styles`