@MikeCook9994
Posted
The first thing that stands out to me is the over-use of pixel units, particularly for font sizes, padding, and margins. By no means, should you never use pixel, but using rem allows for more accessible designs that automatically adopt and adapt the user's text sizing preferences.
You set an explicit height on your container. Which this might help achieve a more exact look, this is generally a bad idea. Generally, you should let the content determine the height of the element so you can prevent overflow.
I think it's probably a good idea if you don't worry too much about trying to create a solution that exactly matches the design. Focus on the fundamentals like responsiveness, the value of certain units, different layout mechanisms, etc...
One last thing, look into the gap property if you're not already familiar with it. It can be used to replace margins in a lot of cases.
Marked as helpful
@BluffSet7340
Posted
I appreciate the feedback. You've cleared my doubts on proper sizing of text. Also, I completely forgot about the gap property too. I'll be sure to implement this feedback in my later solutions.
