Skip to content
OverviewSolutions
3
Comments
3
Darren Draper
@darren-2016

All comments

  • Darren Draper•50
    @darren-2016Posted 4 months ago

    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
  • Darren Draper•50
    @darren-2016Posted 4 months ago

    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
  • Javier Alberto Vásquez Suárez•470
    @JaviVaSuSubmitted 7 months ago
    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?

    .

    Responsive QR Code Component

    1
    Darren Draper•50
    @darren-2016Posted 7 months ago

    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