@Diarrah
Posted
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.
Looking to hire developers?
Feedback, Please?
@Diarrah
Posted
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.
@mattstuddert
Posted
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! 👍
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