@Mel1207
Posted
Hi Rachel! great work on mobile layout 😀
here are some issue I found:
-
Layout for desktop (1430px+ and higher): Text and background are not properly aligned based on the mockup you can try check them on your project files.
-
Missing triangular point at the top of your toggle menu. you can check that on google just search css triangle and add it on ::before or ::after of your toggle menu.
-
You can add transition from hovers to make some smooth animations just for UI enhancement.
- You can use overflow-x property to hide horizontal scrollbar on your page.
you can check my [solution] (https://www.frontendmentor.io/solutions/html-sass-and-gsap-for-basic-animations-UBpUkE2Sb#feedback) for reference.
hope this would help. please upvote my comment if it does, thank you. happy coding 😃
Marked as helpful
@rboz1
Posted
@Mel1207 Thank you so much for the feedback!! I didn't even think to design for screens larger than mine!