@vanzasetia
Posted
Hi, Nafsuki! 👋
Good effort on this challenge! 👍
Just to let you know that I don't understand Next.js. So, I give feedback based on the HTML markup from the developer tool.
I like the way you handle the issue by adding a button where the user on mobile view can scroll down. But, there are some issues with it.
- It's not an interactive element. I would expect it as an anchor tag that links to the next section.
- There's no text content in it. So, the screenreader users would not know there's a link.
For the second section of the home page, I recommend using grid instead of flexbox. Grid is the best option because the layout requires the column and the row. Flexbox is good for a one-direction layout (either column or row).
Hope this helps. 😉
Marked as helpful
@Nafsuki
Posted
@vanzasetia Hi👋🏻 Thank you very much for the constructive advice! I agree! I'll keep that in mind for the next project😊 It means for me a lot! Thank you!!
@vanzasetia
Posted
@Nafsuki No problem! 👍