search

pressbooks / pressbooks-book

McLuhan is the default book theme for Pressbooks.
https://pressbooks.org
GNU General Public License v3.0
20 stars 16 forks source link

Restore expected list-styling for <details> element #746

Closed SteelWagstaff closed 3 years ago

SteelWagstaff commented 3 years ago

Description

Aetna currently includes a CSS rule which suppresses the expected show/hide caret for details summary elements.

https://github.com/pressbooks/pressbooks-book/blob/527f858dde3f524bffa6dbb2da568a448d9f088d/packages/aetna/assets/styles/common/_reboot.scss#L508-L510

As a result users are often unaware (visually) that the details element is present and contains an invisible disclosure block.

https://user-images.githubusercontent.com/13485451/113320080-f8c87a00-92c6-11eb-8624-1c4a84fa1746.mp4

We should remove the offending CSS so that the details element includes the expected show/hide list style caret.

See https://twitter.com/cogdog/status/1377643536867151874?s=20 for initial report from user.

See https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details for more information on styling options for the <details> and summary elements.

SteelWagstaff commented 3 years ago

Tested and working as expected on integrations