Design comparison
SolutionDesign
Solution retrospective
I didn't find the project overly hard, however I feel like my CSS could be a bit more condense? Any advice on improvements will be greatly appreciated!
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @CalebWhorton12!
Your solution looks excellent!
I have just one suggestion:
- Since the title is a clickable element, it's a good practice to add
cursor: pointer;
to it. It gives the user a visual indication that's clickable.
And about your CSS code, I took a look and it looks pretty good. Very well organized. I'd just use the variables a bit more. On a project like this, you don't need many, but it's nice to get used to them, because as you progress, you'll need it more and more.
I hope it helps!
Other than this little detail, you did a really good job!
Marked as helpful0 - Since the title is a clickable element, it's a good practice to add
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord