-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
🐛 Bug: Navbar not active on tools page. #1375
Comments
Hey @cbum-dev , I hope you doing well, can you elaborate more about this issue , what should be expected there, I have seen your providing video, i am not able to understand that |
Like its related to navbar active state. Clicking on tooling page will not activate the navbar "eg: not underlining the tools navigation" |
Ok , I got it |
Navbar is inactive in light mode for all links. while in dark mode for tools link. |
Hey @DhairyaMajmudar, I hope you're doing well. I would love to work on this issue if it's possible. Could you please assign it to me? |
@jagpreetrahi I have already fixed this issue. Maybe you should work on some other issues. But I would love to see your approach towards this issue. 🚀 |
Hey @cbum-dev, amazing resolving the issue so quickly , and it’s been a pleasure understanding the issue alongside you. |
Its been 3 days since I opened it 😅😅. But thanks. |
@DhairyaMajmudar Please assign this to me. |
Hey @cbum-dev , can you please provide me the details where you exactly found that bug inside the file, bcz you told that like already had fixed issue. |
@DhairyaMajmudar Can you please assign it to me as this is an "accepted" issue. |
Mainly on the navbar component and some other files. |
Describe the bug
While navigating to tools page and its related sub-page the navbar is not active.
The pages under blogs and other pages should also make the navbar active with "/blog".
Steps To Reproduce
Go to https://json-schema.org/tools and see the navbar
Expected Behavior
The tools should be in active state.
Screenshots
Screencast.From.2025-01-25.17-57-06.mp4
Device Information [optional]
Are you working on this issue?
Yes
Do you think this work might require an [Architectural Decision Record (ADR)]? (significant or noteworthy)
Yes
The text was updated successfully, but these errors were encountered: