P
ApplePieGiraffe• 30,545
@ApplePieGiraffe
Posted
Hey, ah298! 👋
Nice work! 👍
I suggest,
- Adding some
alt
text to your images for accessibility purposes. - Perhaps switching to a mobile-friendly layout a little sooner than 414px so that the content of the page doesn't look too squeezed right before the layout changes.
- Your HTML looks organized! I think you should turn the "FAQ" title into a heading, however, in order to make your HTML more semantic.
- I see you organized your Sass into different files, which is a good practice, too! Your comments are easy to read and understand (which is nice).
Keep coding (and happy coding, too)! 😁
0
ah298• 165
@ah298
Posted
@ApplePieGiraffe thank u so much for the feedback. really appreciate it! :D
0