-
Notifications
You must be signed in to change notification settings - Fork 66
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
Updates to Academy colors and fonts #1774
Labels
Comments
No, these need a fix, I appreciate your pointing them out and I'll create a backlog item with these. The code heading really needs to come back as the most urgent item. Thanks for this, Michelle! I really appreciated your above-and-beyond attitude and your cheerful competence during your time with us this summer. Hope we get to work together again. |
Merged
SharpRake
pushed a commit
that referenced
this issue
Sep 4, 2024
- Fix visibility of content type label for code blocks - Update code block colors to match images.chainguard Related to #1774
erikaheidi
pushed a commit
that referenced
this issue
Sep 19, 2024
- 🧹 Fix table background color in dark mode - 🧹 Fix navigation/footer/card/button border color - 🧹 Fix navigation active menu item background color Related #1774
erikaheidi
pushed a commit
that referenced
this issue
Sep 26, 2024
- 🧹 Updated `code` background color - 🧹 Updated tags and pills background color - 🧹 Fix `blockquote` background color Closes #1774 (Updating rumble charts is the only outstanding thing from this issue)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey, @smythp !
Unfortunately, I won't have a chance to tackle this before I head out today. I'm going to compile a list of things that I noticed on the Academy which might warrant an update. I suspect these are lingering from the old site theme.
Colors:
In dark mode, the tags, monospaced text, and selected article colors are still the same dark grey color from the old theme.
A similar case is seen with the copy-and-paste code blocks in our tutorials:
Same is true for tables:
There are thin, grey bars that remain from the old color theme. Also around the prev->next buttons. Super hard to pick out, but I think they should be neon purple.
Chart colors could do with a refresh...
There are also these numbered blocks on this article, https://edu.chainguard.dev/software-security/cves/cve-remediation/
Fonts / Text:
Code blocks no longer have text explaining what type of content they are. There used to be a "heading" to each code block with the content type (ex. sh, Dockerfile, Python, etc.) These content types were pulled from the markdown:
```Dockerfile
....
or
``` Python
...
I could have sworn that a default font was displaying somewhere, but I think it was in the case of the above example... so if that functionality is gone, then the incorrect font went with it.
You've done such an incredible job on this effort, I by no means am trying to be nitpicky or critical of your work. I would have loved to jump on these changes if I had another week. The color palette thing is super tricky now that we're working with a purpleish base color, especially on dark mode. I strongly feel that the color palette needs to be expanded so that you have more flexibility. The old theme had many shades of grey that helped with these changes, but we've been using the secondary and tertiary colors with much difficulty. 🤷 just my thoughts on that.
Do what you wish with these suggestions! Do not hesitate to counter them, ignore them, etc. I'm mainly dumping this here for tracking since I won't have the opportunity to work on it myself.
The text was updated successfully, but these errors were encountered: