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

Order summary component with HTML and CSS

Alaisah Abimβ€’ 120

@Abimimbom

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


All feedback and critiques are highly welcome.

Community feedback

Panjiβ€’ 2,110

@pperdana

Posted

Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

  • I have some additional recommendations for your code that I think you'll find interesting and valuable.

πŸ“Œ Add <footer> tag as semantic HTML in code

  • The <footer> tag is another semantic HTML element that is used to define the footer section of a web page.

  • The <footer> tag should be used to wrap the content that appears at the bottom of a web page, such as copyright notices, contact information, or links to other pages.

for example code:

<footer class="attribution">
  Challenge by <a href="https://www.frontendmentor.io?ref=challenge" 
  target="_blank">Frontend Mentor</a>. 
  .................................
</footer>

In the example above, the <footer> tag is used to wrap the copyright notice, which is the content that appears at the bottom of the web page. This tells both human readers and search engines that the content inside the <footer> tag is the footer section of the web page.

I hope you found this helpful!

Happy codingπŸ€–

1

Alaisah Abimβ€’ 120

@Abimimbom

Posted

THANK YOU so much. I'll be using more semantic html from now on @Panji200

0
dimar hanungβ€’ 560

@dimar-hanung

Posted

Hello! πŸ–οΈ Well done on completing the challenge – you did it! 🌟

I have some interest and feedback with your code

That i like:

  1. I appreciate the similarity of your results with the design, a bit different in scale but still good
  2. html is pretty good, not too nested with combination semantic HTML πŸ‘
  3. CSS Naming is also good, represent what is it for, likeΒ <div class="plan">Β for plan section

My Feedback:

  • Is not too responsive on screen < 380px, to fix it you can add flex-wrap: wrap in .plan

  • you can seperate file by folder to be more structured, for example:

    public/
    β”œβ”€ images/
    β”‚  β”œβ”€ pattern-background-desktop.svg // and etc
    β”œβ”€ styles/
    β”‚  β”œβ”€ main.css
    β”œβ”€ favicon.png
    index.html
    
    

In summary, great work and an impressive solution - keep it up!, hope it's helpful πŸ‘

1

Alaisah Abimβ€’ 120

@Abimimbom

Posted

Thank you very much for your feedback. I'll make changes to the code as per your recommendation. Much appreciated @dimar-hanung

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