Skip to content
OverviewSolutions
3
Comments
4
Colt
@Xerver8694

All comments

  • Colt•90
    @Xerver8694Posted 2 days ago

    The code looks very easy to read through and debug, and the solution looks pretty much identical to the solution. Personally, I would have used buttons for the social links, but the way you have coded the list of links also works. If anything, it might actually be better than using buttons. Overall, excellent job. Happy coding!

    Marked as helpful
    1
  • Christina A. Pascual•10
    @devchris03Submitted 4 days ago
    What are you most proud of, and what would you do differently next time?

    To have been able to complete my first challenge in a short time. I believe that for the next challenge I will be able to make the process of uploading the changes and the solution of the project in a faster way.

    What challenges did you encounter, and how did you overcome them?

    So far, none. The challenge was based on the knowledge I have.

    What specific areas of your project would you like help with?

    Could you recommend me good practices to establish styles in a more orderly way?

    Solution with flex for qr code

    1
    Colt•90
    @Xerver8694Posted 4 days ago

    The solution matches the design perfectly! The HTML is very readable and CSS is very easy to follow as well. The only thing I can say is that all of the links in the head of the index are not entirely necessary to complete the solution. Everything needed is provided in the project files. Overall, a good solution, but could be made simpler.

    Marked as helpful
    0
  • Colt•90
    @Xerver8694Posted 6 days ago

    The HTML is very easy to read through and understand. It also appears to be accessible, as far as I can tell. The preview site looks good as well, but the styling is a little bit off from the original design, the shadows could use some more work. I struggle with CSS myself, so I am not one to talk though. Aside from that, the solution appears to meet almost of the marks.

    Marked as helpful
    0
  • Colt•90
    @Xerver8694Posted 7 days ago

    This solution looks fantastic. The code is very easy to read through and appears to be accessible to screen readers. The layout looks good as well, and the code is structured in a nice way so as to be reuseable.

    0