scss/_typography.scss declares two contradicting `details` rule blocks back-to-back
#438 opened on Jun 22, 2026
Repository metrics
- Stars
- (1,804 stars)
- PR merge metrics
- (PR metrics pending)
Description
scss/_typography.scss declares the details selector twice back-to-back: once at lines 66-77 and again at lines 79-99. The two blocks restate the same summary and details[open] rules with different values for padding, font-weight, margin-bottom, and add a margin-left in the second block that the first one never had. Source order wins, so the first block's padding: (.25 * $em) (.5 * $em) and font-weight: 600 for details summary are silently overridden by the second block's padding: .5 * $em and font-weight: bold, and the first details[open] summary { margin-bottom: .5 * $em } is replaced by the second block's nested form. Both copies still ship in dist/tacit-css.css and dist/tacit-css.min.css because nothing in the build collapses duplicate selectors with conflicting bodies, so the bundle carries dead bytes and any reader of the source has to guess which block describes the real shipped styling.
The smallest fix is to delete one of the two blocks and keep the values that match the rendered look. The second block reads like the more recent edit (it adds margin-left: -.5 * $em and padding-bottom: .5 * $em and switches font-weight to the keyword bold), so the first block at lines 66-77 is the most likely leftover. Confirm visually against index.html, then drop the stale block in scss/_typography.scss so a single details rule survives.