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

Fix #1840. Restore glossary page rendering. #1841

Merged
merged 2 commits into from
Feb 14, 2025
Merged

Fix #1840. Restore glossary page rendering. #1841

merged 2 commits into from
Feb 14, 2025

Conversation

scottmakestech
Copy link
Collaborator

No description provided.

@scottmakestech
Copy link
Collaborator Author

@bdaehlie Here is the updated glossary page. https://deploy-preview-1841--letsencrypt.netlify.app/docs/glossary/

I changed the rendering so it uses the definition list semantic elements <DL>, <DT>, and <DD>. However, that forces the term and definition onto separate lines. The previous style was all on one line: Term: Definition Text lorem ipsum. Are you ok with the new approach?

@bdaehlie
Copy link
Collaborator

Looks good to me, thanks!

@scottmakestech scottmakestech merged commit 7a25b43 into main Feb 14, 2025
5 checks passed
@scottmakestech scottmakestech deleted the issue-1840 branch February 14, 2025 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants