-
Notifications
You must be signed in to change notification settings - Fork 50
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 docs theme #784
Update docs theme #784
Conversation
Signed-off-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
✅ Deploy Preview for conda-store ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Co-authored-by: smeragoel <98317216+smeragoel@users.noreply.github.com>
I've made some color updates and currently, there are no accessibility issues detected, except for a minor one in the breadcrumbs. To address this, I recommend the following change: From: To: Also, I want to update the footer colour: I'm not entirely sure which file needs this adjustment, so I'm hoping you could help out with this, @pavithraes. Once this change is implemented, I plan to do a final check to ensure everything is in order, and then we should be all set! |
Signed-off-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
Signed-off-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
@smeragoel - The changes you requested are done :) |
@pavithraes the netlify preview isn't working for me, could you help out? |
@smeragoel - Preview is available now :) |
Signed-off-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
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.
Looks great! There don't seem to be any major accessibility issues, however, I do think we could benefit from an a11y audit down the line.
Fixes #764
Description
This pull request updates the documentation theme properties with the conda-store theme.
Currently, this is a draft PR to have a preview as we make updates.
Pull request checklist
Additional information
Add @smeragoel as a co-author while merging.
How to test