Lighter Bold #8777
Merged
Lighter Bold #8777
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for ilios-frontend ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
dartajax
approved these changes
Aug 18, 2025
Member
dartajax
left a comment
dartajax
left a comment
There was a problem hiding this comment.
Interesting - a lot lighter for sure but I like it - not yelling as much for sure.
Member
Author
|
Team Discussion - overall, good. However, in areas where user's in put data (such as session description) we should hold on to the darker bold. It is simply too light to differentiate in this content (such as in a single event view). |
Locking down on our font-weights, start with declaring that bold is now
600. Usually it is 700 by default, but we have a lot of places where
we've already set the weight at 600 so I matched that.
These are the same thing, but being more explicit helps with linting.
Standardizing on 600 for bold
Remove the option to use the confusing names and require a specific set of font weights.
Removed the font-weights we don't support
I've removed the local aliases as we're preloading these anyway so there is no gain if we pull a local copy and also modernized the src as every browser we support supports woff2 and the format syntax.
I was a little to hasty to cut these, they add nothing to our load times, but finding them again once they're gone will be annoying. Restored.
Locking in a default here, the one we've been using was 700 out of normalize.css
This is slightly less bold now which lowered the contrast against the blue background. This darker orange is more in line with our current colors anyway and meets a11y requirements.
When user input in a session description or objective title is shown to students we want it to be very bold. There is a substantial difference in appearance between 600 and 700 font weight here so I've made an exception and pushed this back to 700.
Member
Author
|
Bold is now bolder for student views of session description and objectives. |
stopfstedt
approved these changes
Aug 28, 2025
jrjohnson
added
a11y
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've locked in a value for our boldest font at 600. Previously we had many at 600, some defaults at 700, and some places with a mix. Hoping this will help with #8773 (review) needs review and discussion.