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

Problem: BS5 theme has typography issues and inconsistencies #1698

Closed
anvit opened this issue Nov 2, 2023 · 0 comments · Fixed by #1699
Closed

Problem: BS5 theme has typography issues and inconsistencies #1698

anvit opened this issue Nov 2, 2023 · 0 comments · Fixed by #1699
Assignees
Labels
Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result.
Milestone

Comments

@anvit
Copy link
Contributor

anvit commented Nov 2, 2023

The BS5 theme has some missing styles due to using reboot.scss from Bootstrap, which exists to remove the default styles that browsers add to HTML tags. This can be desirable while building a new theme from the ground up, but in our case has lead to the blockquote, code, pre tags missing some essential styling.

A few examples of this can be found when using markdown. Here is what the BS2 version looks like
BS2-markdown

Compare this to the BS5 version
BS5-markdown-current

Another issue that comes from reboot.scss is that the default sans serif typeface is system-ui. This leads to different users having widely different experiences, for instance: most current Windows versions default to Segoe UI typeface which doesn't ship with a 500 weight font, and as a result they won't see any sections with the <strong> tag with having any kind of emphasis applied:
BS5-home-current-windows

Here is what it looks like on Ubuntu 22.04 for comparison:
BS5-home-current-ubuntu22

@anvit anvit changed the title Fix BS5 theming inconsistencies (font-weight in windows, blockquote styling missing due to reboot.scss) Problem: BS5 theme has typography issues and inconsistencies Nov 2, 2023
anvit added a commit that referenced this issue Nov 2, 2023
Added the Open Sans typeface as set it as the default sans-serif
typeface for the BS5 theme.
Also added the following style changes:
- Left border and padding for blockquote tags.
- Background color for code and pre tags.
- Padding for code and pre tags.
- Whitespace wrapping for code tags.
- Set the default font-weight for bold and strong tags to 700
@anvit anvit linked a pull request Nov 2, 2023 that will close this issue
anvit added a commit that referenced this issue Nov 2, 2023
Added the Open Sans typeface as set it as the default sans-serif
typeface for the BS5 theme.
Also added the following style changes:
- Left border and padding for blockquote tags.
- Background color for code and pre tags.
- Padding for code and pre tags.
- Whitespace wrapping for code tags.
- Set the default font-weight for bold and strong tags to 700
@anvit anvit self-assigned this Nov 2, 2023
@anvit anvit added this to the 2.8.0 milestone Nov 2, 2023
anvit added a commit that referenced this issue Nov 3, 2023
Added the Open Sans typeface as set it as the default sans-serif
typeface for the BS5 theme.
Also added the following style changes:
- Left border and padding for blockquote tags.
- Background color for code and pre tags.
- Padding for code and pre tags.
- Whitespace wrapping for code tags.
- Set the default font-weight for bold and strong tags to 700
anvit added a commit that referenced this issue Nov 8, 2023
Added the Open Sans typeface as set it as the default sans-serif
typeface for the BS5 theme.
Also added the following style changes:
- Left border and padding for blockquote tags.
- Background color for code and pre tags.
- Padding for code and pre tags.
- Whitespace wrapping for code tags.
- Set the default font-weight for bold text to 700
anvit added a commit that referenced this issue Nov 8, 2023
Added the Open Sans typeface as set it as the default sans-serif
typeface for the BS5 theme.
Also added the following style changes:
- Left border and padding for blockquote tags.
- Background color for code and pre tags.
- Padding for code and pre tags.
- Whitespace wrapping for code tags.
- Set the default font-weight for bold text to 700
@anvit anvit added the Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result. label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant