Skip to content
OverviewSolutions
5
Comments
3
IanV
@ivdevelopment

All comments

  • IanV•90
    @ivdevelopmentPosted over 2 years ago

    Hi nice work!

    For more semantic html you should use the section and header tag. This will make the code more readable :)

    Hope this helps, happy coding!

    Marked as helpful
    0
  • Priya Garg•200
    @Priya366Submitted over 2 years ago

    This is my first project on frontend mentor which I have completed without watching any tutorial .It took me one and half hour to build this project. I find it difficult in making mobile first layouts . I would love to have your feedbacks on ''How to make a card responsive for a screen size below 350px width?"

    Responsive Card using CSS Flexbox

    1
    IanV•90
    @ivdevelopmentPosted over 2 years ago

    Nice work!

    I would make a folder called css and put the style.css in that folder just to make it more organised.

    Also you forgot to add the font to your solution :)

    But overall great work, keep training!

    Marked as helpful
    0
  • IanV•90
    @ivdevelopmentSubmitted over 2 years ago

    I had more work on this than I thought, did I make it more complex than it actually is?

    NFT Preview Card Component

    2
    IanV•90
    @ivdevelopmentPosted over 2 years ago

    I just updated the solution, I've added hover effects, however I'm not entirely sure how I can add the eye to the image when I hover over it.. Anyone that could help me with this? it's much appreciated!

    Also the screenshot of the design I made is broken, I'm not sure why this happened.

    0