Design comparison
Solution retrospective
Feedback, Please?
Community feedback
- @DiarrahPosted over 4 years ago
Pretty spot on so nice job on that!
Looking at your code, I would take out all that unnecessary CSS. And one more small thing: add a cursor:pointer for your button so that it's more obvious that it's a call to action.
1 - @mattstuddertPosted over 4 years ago
Nice work, Nasratullah! At the moment, you're not using any heading elements. So I'd recommend reviewing your HTML and adding appropriate heading levels to create a content hierarchy. This is especially important when it comes to building accessible websites, as screen reader software will use heading levels to create the content hierarchy.
Visually it looks great though. Keep it up! 👍
0
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