Skip to content
Submitted over 3 years ago

single-price-grid-component- SOLUTION

cosmin888•40
@cosmin870
A solution to the Single price grid component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hi! i would like some feedback for this project. This is my first ever project , i still have a lot to learn please comment your suggestions! Thank you in advance.

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Heli0s•670
    @zeerobit

    • in your html i would use the main tag instead of div for the grid class so you avoid getting any accessibility issue also it's more semantic .

    • I would suggest to rename your div1, div2, div3 classes with something more meaningful for instance class ="community" , class="subscription" etc...

    • For the signup button you could make it an anchor tag instead of using the button element since it's supposed to be a link that redirect you to another page. The button element fits better where the user has to submit a form

    • for the why us section you should use an unorganized list instead of the paragraph elements

    by the way the paragraphs and the list items don't need to be in bold

    Overall you've done well the page looks good .. Also i'm new to coding just sharing what i've learned hope it helps

    Marked as helpful

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