-
Notifications
You must be signed in to change notification settings - Fork 202
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
Change text to secondary on content-heavy pages #5148
base: main
Are you sure you want to change the base?
Conversation
Latest k6 run output1
Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great 🚀
Thanks @fcoveram! I'll update the visual regression testing snapshots then. |
I created this ticket requesting the update of |
@fcoveram changing the For example on this page the search query, artist name, duration, genre/category and license icons have all gotten darker. |
c079521
to
a091131
Compare
Fixes
Fixes #5119 by @obulat
Fixes #5151 by @fcoveram
Description
This PR changes the content pages to use
text-secondary
for the body text, while retainingtext-default
for the headings.Testing Instructions
Visit the content pages and see that the text should be a little lighter and less contrasted with the background.
I'll update the snapshots once @fcoveram approves the new designs.
Checklist
Update index.md
).main
) or a parent feature branch.ov just catalog/generate-docs
for catalogPRs) or the media properties generator (
ov just catalog/generate-docs media-props
for the catalog or
ov just api/generate-docs
for the API) where applicable.Developer Certificate of Origin
Developer Certificate of Origin