Manpreet Singh• 190
@Manpreet-01
Submitted
Darren Draper
@darren-2016
All comments
- Darren Draper• 50
@darren-2016
Posted
This solution looks good, replicating the layout and style of the original design. The implementation works well on a range of screen sizes.
Unable to comment on the code as the View Code button doesn't appear to link to the relevant repo.
Great work!
0 - Bhavy Jogani• 20
@bhavy-j
Submitted
Darren Draper• 50@darren-2016
Posted
The code is well structured. It might be improved by moving the style information into a separate stylesheet CSS file. This has the benefit of more maintainable and readable code, particularly in projects that are likely to grow.
The solution closely matches the design in most aspects. The'HTML & CSS foundations' heading should ideally match in colour to the design.
The card dimensions could also be tweaked to match the size of the design.
Overall a nice solution.
1 @JaviVaSu
Submitted
What are you most proud of, and what would you do differently next time?
.
What challenges did you encounter, and how did you overcome them?
.
What specific areas of your project would you like help with?
.
Darren Draper• 50@darren-2016
Posted
Good clear design and implementation. The source code is well structured, both the home page (index.html) and the stylesheet (style.css). It's clear to follow, and concise. The separation of concerns is well implemented, with index.html handling the content and structure, and style.css defining the presentation.
The layout is responsive when tested using the Developer Tools across various viewport sizes, and the look of the component closely reflects the original design.
Well done Javier on a great implementation!
Marked as helpful
0
