Closed iamjamesfrancis closed 1 year ago
Also, I accidentally fixed a UI issue while doing this issue. The UI issue was we needed to click the icon or text to navigate. Now we can click the entire area😍.
Before
Now
Pull request accepted! Super good job!!
What does the PR do?
Fixes #58
Changes
.sidebar-text-wrapper
from the anchor tag of sidebar list items because it is creating a weird hover..mx-2
from the<font-awesome-icon>
tags and replaced the spacing using.gap-2
.nav-link
from the<a>
infrontend/src/components/Navbar.vue
because itsdisplay: block
is affecting the propertydisplay: flex
..flex
and.items-center
infrontend/src/style.css
.Screenshots
Mobile View
https://user-images.githubusercontent.com/55175085/235984310-436af8aa-f680-4bfe-897b-a540b8dfb4f6.mp4
Desktop View
https://user-images.githubusercontent.com/55175085/235984570-e73b10a9-24ca-41d0-9eff-1cc4de6793ce.mp4