@TedJenkler
Posted
Hi @Jan-Dev0,
Nice job on the project! I noticed a small issue with responsiveness. I’d recommend always using rem or % for sizing, and avoiding fixed widths—use max-width and min-width instead. Like mentioned in previous feedback, the easiest way to tackle this challenge is by using Grid, but it’s totally doable with Flexbox if you play around with media queries. You could use three flex parents, though Grid is generally recommended for more control.
On several breakpoints, there are width and height inconsistencies. I’d suggest looking into that and trying to use as little CSS as possible—it often simplifies things.
To improve SEO and accessibility, consider using <section> tags instead of <div>, with aria-label, aria-labelledby, or title attributes to make the content more accessible.
Your code is heavily nested with <div> elements. For the next project, I’d recommend minimizing the use of <div>s to keep the code cleaner and more readable. For instance, using flex-column or flex-row instead of extra <div> containers can help organize and stack elements more efficiently.
Keep up the great work!
Best, Teodor
Marked as helpful