nash12• 20
@nash4826
Submitted
It seems that the <div> tag was used excessively.
IanV
@ivdevelopment
All comments
- IanV• 90
@ivdevelopment
Posted
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
@Priya366
Submitted
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?"
IanV• 90@ivdevelopment
Posted
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
@ivdevelopment
Submitted
I had more work on this than I thought, did I make it more complex than it actually is?
IanV• 90@ivdevelopment
Posted
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
