Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @tttam0113

    Posted

    • I understand that you only implemented the CSS based on the image, but you should try to use parameters that match the design as closely as possible.
    • You have only designed based on the desktop version, so you should update the page to make it responsive for mobile mode as well.
    • When hovering over the "li" tags, you should also change the text color.
    0
  • @tttam0113

    Posted

    Great work. I like the way you use HTML semantic tags.

    0
  • @tttam0113

    Posted

    It looks great but in my opinion:

    • page should have one main landmark by using main tag for the container
    • update shadow to same design parameter. This is mine: box-shadow: 0 25px 25px rgba(0, 0, 0, 0.05);
    0