From c884d4853fae6e1086b2c5d1646d2af2a16060f6 Mon Sep 17 00:00:00 2001 From: David Apirian Date: Tue, 1 Sep 2026 20:57:53 -0700 Subject: [PATCH] Set markdown hr thickness to border-width (1px) The markdown thematic break (hr) rendered at 4px via the $em-spacer-3 spacing token, roughly 4x the ~1px used by every other markdown renderer and by Primer's own rules (headings, tables, and blockquote all use $border-width). Use $border-width for the hr so it matches the system's line weight and tracks the token if it ever changes. Only the existing $border-width token is used; no new tokens or API. Passes stylelint with no new disables. --- .changeset/markdown-hr-thickness.md | 5 +++++ src/markdown/markdown-body.scss | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) create mode 100644 .changeset/markdown-hr-thickness.md diff --git a/.changeset/markdown-hr-thickness.md b/.changeset/markdown-hr-thickness.md new file mode 100644 index 0000000000..2a415fb05b --- /dev/null +++ b/.changeset/markdown-hr-thickness.md @@ -0,0 +1,5 @@ +--- +"@primer/css": patch +--- + +Set the markdown `
` (thematic break) thickness to 1px using `$border-width`, down from 4px, so it matches Primer's line weight and the ~1px used by other markdown renderers. diff --git a/src/markdown/markdown-body.scss b/src/markdown/markdown-body.scss index e2050acc1b..27f1b9861f 100644 --- a/src/markdown/markdown-body.scss +++ b/src/markdown/markdown-body.scss @@ -76,7 +76,7 @@ } hr { - height: $em-spacer-3; + height: $border-width; padding: 0; margin: var(--base-size-24) 0; // stylelint-disable-next-line primer/colors