@manuelsclSubmitted 5 months ago
manzii07
@manzii07All comments
- @manzii07Posted 4 months ago
The solution includes semantic HTML elements, which is good for SEO and accessibility. However, it could improve accessibility by adding appropriate ARIA roles and ensuring color contrast is sufficient for readability. The layout is mostly responsive, but some adjustments are needed for smaller screen sizes to ensure all content is accessible without excessive scrolling. The code is generally well-structured and readable, but could benefit from more comments and modularity to enhance reusability. Overall, the solution closely follows the design, though some minor differences are present that could be refined for a more polished final product.
0 - @rosemutaiSubmitted 4 months ago@manzii07Posted 4 months ago
Does the solution include semantic HTML?I don't think so Is it accessible, and what improvements could be made? It's accessible but you could try buttons with links that lead somewhere. Does the layout look good on a range of screen sizes?Think so, yeah. Is the code well-structured, readable, and reusable? Yep Does the solution differ considerably from the design?NO
0 - @rosemutaiSubmitted 4 months ago@manzii07Posted 4 months ago
Does the solution include semantic HTML? I think it does Is it accessible, and what improvements could be made? It's accessible and you can try aligning the solution in the same position as the design Does the layout look good on a range of screen sizes? Absolutely yes Is the code well-structured, readable, and reusable? Yes Does the solution differ considerably from the design? Yh
0 - @jasonS12Submitted over 1 year ago@manzii07Posted 4 months ago
The code seems pretty good, so I can front-end project looks very similar to the one provided so Good job!
0
