bug:Navbar issued resolved #50
Conversation
|
|
||
| .line { | ||
| width: 25px; | ||
| /* Adjust width as needed */ |
There was a problem hiding this comment.
nit: these comments are not needed. Should be removed.
| margin: 5px; | ||
| /* Adjust margin as needed */ | ||
| transition: all 0.3s ease; | ||
| /* Optional: Add transition effect */ |
There was a problem hiding this comment.
nit: these comments are not needed. Should be removed.
| .hamburger { | ||
| display: flex; | ||
| flex-direction: column; | ||
| cursor: pointer; | ||
| } | ||
|
|
||
| .hamburger div { | ||
| /* .hamburger div { |
There was a problem hiding this comment.
nit: why are we commenting out code that is not being used. Should be removed.
| @@ -67,13 +68,13 @@ | |||
| display: none; | |||
| } | |||
|
|
|||
| .hamburger div { | |||
| /* .hamburger div { | |||
There was a problem hiding this comment.
nit: why are we commenting out code that is not being used. Should be removed.
|
Hey @Anureng, I left quite a few comments. These will have to be fixed before the PR can work. Outside of the comments this is still an issue Also if you look here, the menu is very far from the right side of the screen here. It needs to be closer to the side. This is how it is in your PR This is how it was originally Let me know if you have any questions. |
No description provided.