fix: resolve navbar logo not loading on nested pages#220
Merged
MrunaliniPachpute merged 1 commit intoOct 30, 2025
Conversation
Thanks for creating a PR for your Issue!
|
Owner
|
@naitik2004 Also, I noticed you’ve updated only 5 topics — that’s fine for now. |
🎉 Congrats on getting your PR merged in, @naitik2004! 🙌🏼Thanks for your contribution every effort helps improve the project. Looking forward to seeing more from you! 🥳✨ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔖 ### Description
Fixed navbar logo image and homepage link across section pages (SegmentTree/segmentTree.html and others, as requested).
Replaced relative paths with absolute paths for the logo image (/Images/logo.png) and homepage link (/index.html) to resolve issues where:
The logo was not displaying after deployment
Clicking the logo caused a 404 "not found" error
These issues were present due to relative paths failing in deployed environments.
Everything worked fine on localhost.
Fixes: # (please enter the related issue number if you have one)
📸 Screenshots (if applicable)
✅ ### Checklist
[x] My code follows the project’s guidelines and style.
[x] I have commented my code where necessary.
[ ] I have updated the documentation if needed.
[x] I have tested the changes and confirmed they work as expected.
[] My PR is linked to a GitHub issue.
🙌 ### Additional Notes
Learned about path sensitivity differences between local and deployed environments.
This fix ensures the navbar is robust and works on all environments.
If other sections/pages use the same navbar, this fix should be applied similarly for consistency.