1- Assuming the actual width and properties of elements was a bit challenging. 2- I am unsure whether I have written an efficient code or not. 3- There are couple of doubts about best practices and whether I am practicing them or not.
@marianarainha
Posted
Hello Shraddha !
You should set hover and focus states for interactive elements, like the button, to change it's background color and the cursor, for exemple.
Marked as helpful
@Shraddha-V22
Posted
@marianarainha thank you. I will do that.
@suhaib52
Posted
Your code is looking clean. It would be great if you use classes as compared to the ids in the html, because use of classes considered best practice as compared to the ids.
Marked as helpful
@Shraddha-V22
Posted
@suhaib52 Yes, I will remember that. thank you!
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
