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

Ferrari 130

@pferrarip

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

hitmorecode 6,230

@hitmorecode

Posted

Nice well done, I have a few pointer

  • Inside .top you have a div inside that div you have two p tags. You gave both p tags an id="txt", this is something you must never do. An id is a unique identifier an id can only occur once on a page.
  • The use of the second p tag is not necessary, you can place the entire content inside one p tag.
  • You used two h1 tags on this page, this is also something you should never do. You can only have one h1 on a page. This has to do with the way search engines work. Just do a search about this.
  • On the last card you used a p for all those descriptions. It's hard to see what the order is, because the position of the content depends on the width of the container. In this case it would be better do use a p tag for each description or you can use ul with a few li for each description.
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