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

Single-Price-NoGrid-Component

@AngelG-JAPY

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


What modification would you make to my code, apart from using grid? Please feedback :P

Community feedback

P
Chamu 12,970

@ChamuMutezva

Posted

Greetings

  • the first heading element of your site has to be an h1 .
  • semantic elements such as header, main, footer should be considered ahead of divs. I would have used the div with class container to be the mainelement.
  • as mentioned by Helios, the Why ussection should be an unordered list.

Happy coding

Marked as helpful

0
Heli0s 670

@zeerobit

Posted

  • the why us section should be an unordered list.. you used the <br> tag in the html to break the lines however you should not use html elements to style your page, leave that to css

  • use rem instead of px since it's not scalable

Happy coding...

Marked as helpful

0

@AngelG-JAPY

Posted

@zeerobit Thank you very much, I had not thought of ul

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