Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

CSS Grid

Sensei_Kabotoβ€’ 100

@Sensei-kn

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Dear friends, I am excited to announce that I have completed the coding project. I would greatly appreciate it if you could take some time to review my work and provide your valuable feedback.

Your feedback will help me improve my coding skills and ensure the quality of our projects. You can access the project " https://sensei-kaboto.github.io/PriceGrid/".

Thank you for your time and support.

Best Regards, Daniel Kaboto

Community feedback

Douoβ€’ 940

@Douoo

Posted

Hi there @Sensei-Kaboto, Congrats on finishing this challenge. I saw your solution and I have to say in visual terms, it is top-notch and deserves credit. In terms of your code, I believe you can improve naming your CSS classes. I suggest you check out the CUBE or BEM way of organizing your CSS classes. Going over your source code I also saw that you used inheritance in your CSS to achieve some of the styling - which is not recommended in a professional environment. It is widely recommended that you give each of your styles a class. Lastly, the line height of the body text on mobile might be better adjusted (they're too close). Other than these you did a good job. Keep it up πŸ‘πŸ½

Marked as helpful

0

Sensei_Kabotoβ€’ 100

@Sensei-kn

Posted

πŸ‘Œhey @Douoo, Thanks a lot for your feedback, I really appreciate πŸ™Œ

0

Please log in to post a comment

Log in with GitHub
Discord logo

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