@pikapikamart
Posted
Hey, awesome work on this one. Site looks good in desktop, though you could have used more of the screen's size to display the desktop layout, the mobile layout starts from 1000px going down which seems to much to be used by mobile.
You already got a great feedback, just going to add some little ones:
- The first two text should be using a single
h1
element, both text are just one phrase so you don't need to use separate heading for each one. - Lastly, the
alt
for the icons on each card should be left emptyalt=""
since those are just decoration.
Again, awesome work on this one.
Marked as helpful
@ryancalacsan
Posted
@pikamart thanks for the feedback! I think I was just moving along quickly and for some reason thought to make the text two different elements instead of just styling the line separately. Will definitely make the change. Thanks again!