@vgt3j4d4
Posted
-
Does the solution include semantic HTML? Yes
-
Is it accessible, and what improvements could be made? Yes. Although I see you added
order
css property. I would avoid using it unless it is really necessary but in my experience 99% of the cases you don't need it. Here is more information about it a11y issues using order in grid and flex -
Does the layout look good on a range of screen sizes? On mobile devices it needs some tweaking. Setting a fixed width to the cards would do the trick (you'll need to update the width on tablet/desktop)
-
Is the code well-structured, readable, and reusable? Yes
-
Does the solution differ considerably from the design? Yeah a bit (font sizes, vertical alignment, box shadows, paddings). I also have my solution differing considerably but you might want to check it to cross check yours.
Marked as helpful