-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix interaction styles for the navigation bar and the footer links #642
Fix interaction styles for the navigation bar and the footer links #642
Conversation
✅ Deploy Preview for idrc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
There are a few issues with this— let's discuss when you have time, @chosww.
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.
One change that I can see— thanks for all your work on this, @chosww!
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.
@chosww Thanks for fixing the hover and focus styling!
I noticed that px units are being used in the SCSS. Previously we have tried to use relative values (rem, vw, vh, etc.) where possible. Can the PR be updated to use rems etc?
npm run test
without errorsnpm run build
without errorsDescription
I noticed that focus and hover styles for the items on navigation bar is broken in different themes. I did some investigation and found out that UIO applies different border colours for different themes and we didn't set any border on the navigation bar items, and that caused no focus and hover styles on them.
An approach I took for the nav bar items are to set transparent border on them all the time by overriding UIO styles, and on focus and hover states, apply correct UIO styles. This way, we can utilize UIO. A potential approach would be not to use UIO, and use outline for the focus and hover styles, but that would be a lot of custom css styles we may maintain in the future as UIO evolves.
I also noticed that focus and hover styles for items on the footer are off, so I made some adjustment to them.
Steps to test
Expected behavior:
Nav bar items should have proper interaction styles in different themes changed by UIO.