Roc Tanweer• 2,500
@RocTanweer
Posted
Hello 👋 You may use header tag to wrap up logo
1
William Luft• 160
@wdluft
Posted
@RocTanweer 👍
0
Design comparison
Solution retrospective
I would love to hear recommendations for what semantic HTML tags I could've used.
Community feedback
Samuel Palacios• 615
@samuelpalaciosdev
Posted
Hello, William! 👋
Great job on this challenge!. Your solution looks nice (almost pixel perfect) and it scales pretty well.
I only suggest some things😉:
-
Adding
cursor: pointerto your button. So people would know that's clickable. -
Take care of those accesibility issues. I suggest you to use the [W3C Markup Validation Service] (https://validator.w3.org/).
I really like the way you approach this, keep coding ;)
0
William Luft• 160
@wdluft
Posted
@samuelpalaciosdev Thanks, I'll check it out!
0
Samuel Palacios• 615
@samuelpalaciosdev
Posted
If that has helpful, please upvote my comment ;)
0
Please log in to post a comment
Log in with GitHub
Join 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