-
Notifications
You must be signed in to change notification settings - Fork 18
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
Active navigation link highlighting not working on the website #42
Comments
@shivankacker please see this and let me know |
Is this issue open to be solved? I'd like to work on this issue. Thanks. |
hey @shivankacker could you please this issue? |
@Krishna100604 This might not work with our current design. Maybe lets just add a small dot at the bottom of the link to indicate it is active? |
is this fine? @shivankacker |
@Krishna100604 let's not change the color maybe? Keep it white, but fully opaque. And decrease the space between the dot and text a little? |
|
|
@Krishna100604 hi, the second one looks good to me |
so i will make a pull request then? |
@Krishna100604 yes please go ahead |
@Krishna100604 @shivankacker hey, I have made the first thing asked in the issue, there are some adjustments to make in it, like you sort of get an underline and the color of the text changes when you hover on a nav item. ![]() |
**This is how it is supposed to look
The text was updated successfully, but these errors were encountered: