Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add closing tags for description list terms #1226

Closed
wants to merge 1 commit into from

Conversation

nevans
Copy link
Contributor

@nevans nevans commented Dec 5, 2024

Without the closing tags, the dt elements contain whitespace after the text. This normally isn't a big deal, but does mess some things up, e.g: using ::after with content: ", " in stylesheets.

Without the closing tags, the dt elements contain whitespace after the
text.  This normally isn't a big deal, but does mess some things up,
e.g: using `::after` with `content: ", "` in stylesheets.
@nevans
Copy link
Contributor Author

nevans commented Dec 5, 2024

n.b: this was extracted from #1209. It allows users with custom styles to use ::after with content: ", ", even if that's never added to the default styles.

@st0012
Copy link
Member

st0012 commented Dec 8, 2024

Included in #1209

@st0012 st0012 closed this Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants