@ryyHardy
Posted
I really like the way you wrote your CSS. I can tell you have a good foundation for this stuff. As for what you mentioned:
- I feel like it's usually better to use Google Fonts, but this article has a pretty good summary of your options for loading fonts.
- I don't mind the CSS nesting, but you can do the same thing using a descendant selector or something similar. Try looking at a reference of CSS selectors to see if there are alternatives to nesting because it isn't that widely used yet.
- I think the image is good. All I noticed was that the padding on the card as a whole looks a little large.
- As an alternative to media queries, I would try to mess around with the clamp() function. It's really good in some cases. I don't know for sure, though.
That's all I have to say. Everything else is really good.
Marked as helpful
@Andre-DM
Posted
@ryyHardy Hi there, firstly, thank you for taking the time to leave me feedback! I took a quick look at the article you linked and, as soon as I have time, I will definitely read it carefully since it seems very useful. Same with clamp(), I will definitely do some research on it. Thanks again for the tips!