Flowchart text color - #340
Merged
LauraLangdon merged 2 commits intoJun 30, 2026
Merged
Conversation
From local testing it appears that the CSS overrides the inline style, i.e. the color directive could have been set to purple #800080 and it would still appear as UC-OSPO-Network#222 (off-black) or whatever is defined in cusstom.css for mermaid text. So I removed this so that it is clear that the flowchart text color is derived from the CSS file
✅ Deploy Preview for ucospo-net ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
LauraLangdon
approved these changes
Jun 30, 2026
LauraLangdon
left a comment
LauraLangdon
left a comment
Contributor
There was a problem hiding this comment.
Thanks so much, @scott-huberty! 🙌🏻 I would have thought our a11y checker would have caught this, but maybe it's not looking at charts and so on. 🤔
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
The text inside flowchart on the get involved page ( https://ucospo.net/about/get-involved/ ) is not legible when viewing the site in dark mode:
This PR updates the CSS styling for flowchart text to use a gray color (instead of black), so that the diagram is legible in both light and dark modes! ( e9be4e5 )
I also noticed that the text color was being formatted both in the CSS directory and via inline styling. during local testing, it appears that the CSS overrides the inline color directive, so I removed it to keep things tidy and clear (1cb108a)