Skip to content

Update guidance on <hr> styling#42708

Merged
chrisdavidmills merged 3 commits intomdn:mainfrom
pepelsbey:hr-styling
Jan 9, 2026
Merged

Update guidance on <hr> styling#42708
chrisdavidmills merged 3 commits intomdn:mainfrom
pepelsbey:hr-styling

Conversation

@pepelsbey
Copy link
Member

@pepelsbey pepelsbey commented Jan 9, 2026

Description

Updates guidance on <hr> styling:

  • Removes ambiguous “appropriate CSS”
  • Adds reference to the border-style CSS property

Motivation

To make it clearer what CSS can be used.

Related issues and pull requests

Fixes #42363

@pepelsbey pepelsbey requested a review from a team as a code owner January 9, 2026 14:14
@pepelsbey pepelsbey requested review from chrisdavidmills and removed request for a team January 9, 2026 14:14
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs size/xs [PR only] 0-5 LoC changed labels Jan 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026 •
edited
Loading

Preview URLs

(comment last updated: 2026-01-09 17:53:30)

chrisdavidmills
chrisdavidmills requested changes Jan 9, 2026
View reviewed changes
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pepelsbey I think this is a good idea, although your addition left me with a few questions. For example, surely other properties such as border-color are also important for styling a line, and it sounds like you are saying these are only useful for styling lines drawn on other elements, not <hr>, which can also be customized in this manner. Also, it was still not clear where you might draw these other lines, so I thought it might be good to clarify.

Anyway, the resulting suggestion might be slightly overkill; let me know what you think ;-)

@pepelsbey @chrisdavidmills
Update index.md
3bfab3f 
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
@pepelsbey
Copy link
Member Author

pepelsbey commented Jan 9, 2026

Oh, I absolutely love your addition! I considered expanding the advice but decided to make it lighter. Perfect!

chrisdavidmills
chrisdavidmills approved these changes Jan 9, 2026
View reviewed changes
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, let's get this merged.

@chrisdavidmills chrisdavidmills merged commit 479f72f into mdn:main Jan 9, 2026
8 checks passed
@pepelsbey pepelsbey deleted the hr-styling branch January 9, 2026 19:12
Anurella added a commit to Anurella/content that referenced this pull request Jan 12, 2026
@Anurella
Merge branch 'main' of https://github.com/Anurella/content
9ae3a51 
* 'main' of https://github.com/Anurella/content: (7201 commits)
  Update InterfaceData based on WebRef (mdn#42718)
  fix typo in `PeroformanceResourceTiming.requestStart` (mdn#42724)
  fix: auto-cleanup by bot (mdn#42717)
  fix typo in RTCTransportStats (mdn#42704)
  Fix typo in queuingStrategy variable name (mdn#42706)
  Update guidance on <hr> styling (mdn#42708)
  Fx147: Add relnote for iterator sequencing support (mdn#42713)
  Fix order of arguments in SpeechRecognitionPhrase ctor (mdn#42711)
  Update warnings about event handler attributes (mdn#42347)
  chore(deps-dev): bump fs-extra from 11.3.2 to 11.3.3 (mdn#42468)
  ci(deps): bump actions/download-artifact from 6.0.0 to 7.0.0 (mdn#42404)
  ci(deps): bump actions/upload-artifact from 5.0.0 to 6.0.0 (mdn#42402)
  ci(deps): bump peter-evans/create-pull-request from 7.0.11 to 8.0.0 (mdn#42403)
  Explain value repetition when fewer background values are specified (mdn#42627)
  Add description to background shorthand property (mdn#42695)
  chore(deps): bump @mdn/fred from 1.9.7 to 2.0.0 (mdn#42702)
  chore(deps): bump qs from 6.14.0 to 6.14.1 (mdn#42598)
  Bump node-html-parser from 7.0.1 to 7.0.2 (mdn#42659)
  update documentation to note that Firefox for Android requires CT (mdn#42672)
  TrustedTypes: Function() constructor + eval() (mdn#42462)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrisdavidmills chrisdavidmills chrisdavidmills approved these changes

Assignees

No one assigned

Labels

Content:HTML Hypertext Markup Language docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

HR article uses phrase "appropriate CSS"

2 participants

@pepelsbey @chrisdavidmills