@TedJenkler
Posted
Hi @YuliaLantzberg,
Nice project! I really like the overlay effect. However, I noticed that your site isn’t fully responsive. I recommend looking into the 850px - 998px and 1500px width ranges to ensure it displays correctly across different screen sizes.
Image Positioning: Usually, it’s best to avoid setting fixed heights, but in this project, you should consider locking the image in place. You can achieve this by setting the card to position: relative and the image to position: absolute, then using top, left, and right properties to lock the image onto the card.
Div Usage: Only use <div> elements when necessary to simplify your structure and make the code cleaner. You usually only need <div>s when you need content to change direction; the rest can often be achieved with a single flex column.
Keep up the great work!
Best, Teodor
Marked as helpful
@YuliaLantzberg
Posted
Hi @TedJenkler. Thank you so much for the valuable and thoughtful feedback. I will definitely look into endpoints that you've mentioned and will try it with your recommendation about the image positioning. You are completely right about the divs. It's my weakness and I will try my best to improve myself and to reduce usage of the divs. Wishes, Yulia :-)