@vgt3j4d4
Posted
-
Does the solution include semantic HTML? Yes, definitely!
-
Is it accessible, and what improvements could be made? I believe it is ok.
-
Does the layout look good on a range of screen sizes? Yes, I see it look good not only in mobile and desktop but the grid accomodates itself depending on the dimensions of the viewport. Well done! (I have to admit I only thought on Desktop and Mobile)
-
Is the code well-structured, readable, and reusable? Yes, specially using separately .scss files (partials)
-
Does the solution differ considerably from the design? Nope
- What specific areas of your project would you like help with? I'd like help to understand if I understood well BEM technology. Hwat could I have done differently in order to make it more reusable? Thanks :). I believe it is pretty reusable, not only using scss but the way things were organized. Also, the naming of the css classes, mixins, etc :)
Marked as helpful
@JamesLaviron
Posted
@vgt3j4d4 Thank you for your feedback, it's much appreciated.
