P
Matt Studdert• 13,611
@mattstuddert
Posted
Hey Musole, awesome work on this challenge. Your solution looks great and scales down really well to mobile. It's good to see you using min-width
media queries as well. The only couple of suggestions I'd have relate to your HTML:
- You've got the "Download for..." call-to-actions as
input
elements inside of forms. Whereas you'd expect these to link off to different websites (the app stores), so they would be better as anchor tags. - You're using the
section
element a lot. Asection
is supposed to group a set of related/themed content, whereas adiv
is perfect for smaller groupings of inner content. So where you've got thesection
with the.workflow
class, I'd say that's perfect, but then I'd usediv
elements to group the content inside of that.
Let me know if you have any questions. Keep up the great work! 👍
0