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
Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Today, I realized that my laptop screen is so small, therefore is difficult to me watch my page in resolutions more than 700px of height, I will appreciate your comments.

Community feedback

@VCarames

Posted

Hey @Kishp73, some suggestions to improve you code:

  • Currently your HTML content isnt structured and your using several element incorrectly. I suggest learning more on that. To better structure your content and make it a lot clearer, you want to implement Semantic HTML.

Why use Semantic HTML?

  1. Accessibility: Semantic HTML makes webpages accessible for mobile devices and for people with disabilities as well. This is because screen readers and browsers are able to interpret the code better.
  2. SEO: It improves the website SEO, or Search Engine Optimization, which is the process of increasing the number of people that visit your webpage. With better SEO, search engines are better able to identify the content of your website and weight the most important content appropriately.
  3. Easy to Understand: Semantic HTML also makes the website’s source code easier to read for other web developers.

More Info

[https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Document_and_website_structure] (https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Document_and_website_structure)

https://www.freecodecamp.org/news/semantic-html5-elements/

https://dev.to/poulamic/writing-semantic-html-3436

  • Add the background-size: contain ; property to your Body Element to have it take up the entire screen.

  • Your not using the Alt Tag effectively. You want to describe what the image is; they need to be readable. Assume you’re describing the image/icon to someone.

  • The music icon is decorative so its Alt Tag should be blank.

  • Your "button" and "cancel order" should NOT be wrapped in headings.

  • The "button" was created with the incorrect elements. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.

Happy Coding!

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