Tereza• 605
@sirriah
Posted
Hello, I saw you code. I think that naming components like "img" etc. is not the best idea. Try to look at BEM naming. This is a small project, but in bigger designs you will get lost.
I think, that you can make the image from side to side with 'object-fit: cover' property.
The whole component should have the maximum width set on specific size. If you have large screen then the width is now set on 100% and it looks strange.
The border radius is better to set with px or rem units.
Happy coding 👍
0
Stephen Balliet• 30
@balliets10
Posted
@sirriah Thank you for the critiques, I will look into your suggestions.
0
