@eleswastaken
Posted
Good job!
On the design image you can see the pointer cursors, that means that those elements are interactable. And that usually means that element is either a button or a link. E.g. in this challenge when you try to click the image, you would expect it to take you somewhere so it is a link. Same is with the card title and author name.
For an alignment. A good practice is to put images in a wrapper(link, div, figure), and set a specified width and height. The eye is 48x48 px, so set the width to that. Then you could center align it using flexbox.
On small screen sizes there is no need to worry about hover effects(usually). You change an element using element:hover {}. You could just change the :active state to the :hover.
Good luck!
Marked as helpful
@Eshan01
Posted
Thank you! @eleswastaken for reviewing my project and clearing all of my doubts, now I'll update this project.
