-
Notifications
You must be signed in to change notification settings - Fork 208
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
Update the homepage dark color #4983
Conversation
78511f9
to
cc80665
Compare
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.
Barring any design changes from @fcoveram, I think this looks great!
cc80665
to
bc161e4
Compare
bc161e4
to
ebc00c3
Compare
@obulat looks like potential flake in an audio test |
f9c7946
to
6dedd69
Compare
Latest k6 run output1
Footnotes
|
Most elements look good. But as per this comment, I have pending updating the design file. Once the file is done, do you think it is better to include those changes in this PR or close this one and open a follow-up one? |
I think it would be best to draft this PR, and update it once the design changes are ready. Would that be okay? Could you also ping here when the design changes are ready? |
Sure. No problem! |
3172d05
to
bac5448
Compare
Full-stack documentation: https://docs.openverse.org/_preview/4983 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
8ccebce
to
f030b52
Compare
c2b30ed
to
aa5d0e6
Compare
@fcoveram, I rebased this PR onto the one that fixes the links focus outline colors (Fix link focus rings). That PR fixes the last two problems (Other focus styles). In this PR, I:
If the link focus rings are okay, could you also approve #4958, @fcoveram ? |
815669b
to
628b474
Compare
1f19fb2
to
b57bc0b
Compare
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 👏 ✨ 🚀
b57bc0b
to
d83e182
Compare
* Fix link focus rings * Fix the page links * Update snapshots * Fix focus-slim-offset offset color * Update the homepage dark color * Use border-tx button for search type * Fix search bar focus styles * Fix border and text-secondary colors * Update snapshots * Use inner ring for search button focus ring * Fix the popover border color * Fix close buttons * Fix button borders * Make sure button is hovered in test * Update snapshots
Fixes
Fixes #4936 by @fcoveram
Description
This PR changes the color of the homepage in dark mode to match the mockups. This also changes the color of the generic error page, you can see the updated version in the changed snapshots.
Testing Instructions
Run the app and go to http://localhost:8443
Change the app theme to dark
Check that the colors are correct, or suggest improvements.
Also check the error page by going to http://localhost:8443/non-existent-page. Note that this page's design will need to be updated, but in this PR only the colors are updated to match the home page.
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