Skip to content
Submitted about 1 year ago

CSS Grid

Sensei_Kaboto•100
@Sensei-kn
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Single price grid component
  • HTML
  • CSS
1newbie

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
    @DouooPosted about 1 year ago

    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-knPosted about 1 year ago

    👌hey @Douoo, Thanks a lot for your feedback, I really appreciate 🙌

    0

Please log in to post a comment

Log in with GitHub

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