z-mn• 150
@z-mn
Posted
Well done. The solution looks good except for the font weight of the headings where yours is too low. Looking at your CSS, I would say that you shouldn't really be using pixels to set widths, heights, paddings, margins and gaps - this should be done using rem and em for better responsiveness. You could also make more variables for properties such as font weight and font size as this will make your CSS easier to maintain. Using pixels primarily in your media query is ineffective as this is only tailored to your screen size - it doesn't work as it should on my screen, for instance. Good job and happy coding! :)
Marked as helpful
0
