
Solution retrospective
I started using the BEM syntax. Howerver, i encountered some problems on how to name the small text above the title and teh button. So i used some utility classes. But still, i am not sure if this is the correct solution.
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@Ant105
Hi, sorry, but I can't give you the feedback you requested due to my lack of knowledge. However it seems I need to leave you feedback to progress in the learning path, so I tried to make my feedback useful despite not being able to provide what you want.
-
Your <button> is too big, which puches all the following elements lower than in the design. I recommend exporting elements/frames from Figma and using the Chrome extension PerfectPixel for more precise layout adjustments. I use it often and I find it very helpful.
-
I think you could wrap your <h1> in a <header>.
-
I also don't see a mobile version, though maybe you only intended to create a desktop version.
You wrote very good code, and I hope I didn't waste your time with my comment, and I'm sorry if I said something wrong.
-
- @jiangying-97
good, I did it.
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