Fixed navbar navlinks and positioning of hamburger menu

atharvarekhawar commented 5 months ago

Related Issue

This PR solves issue #203

Description

The Home navlink on index.html lead to wrong page , fixed it by directing it to home.html. The positioning of Hamburger menu on mobile devices was off so grouped it with theme toggle button. The navbar is visible in the mobile view without missing any content.

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): ___

Screenshots / videos (if applicable)

https://github.com/CSEdgeOfficial/learn.csedge.courses/assets/113119063/49327715-ebc1-4482-a60f-04cc0e32a51a

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:

Please add appropriate labels to the PR , Thank you 😊.

github-actions[bot] commented 5 months ago

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

CSEdgeOff commented 5 months ago

@atharvarekhawar Getting conflicts recommit your changes with the new updates and Don't do any changes in the main Index.html if any changes were done revert them

atharvarekhawar commented 5 months ago

@CSEdgeOff done with the changes.

CSEdgeOfficial / learn.csedge.courses