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

HTML, CSS

@tomruzicka

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


Hi, I am a junior developer and I would be grateful for your feedback!

Community feedback

Tereza 605

@sirriah

Posted

Zdravím Tome, 🇨🇿 vítej na FM.

  • zkus příští projekt udělat mobile-first
  • mrkni na BEM naming
  • podívej se na semantic HTML, tahle challenge je malá, ale určitě by se tam daly použít tagy jako header, main a footer :)

Skvělá první challenge 👍 těším se na další!

Tereza

Marked as helpful

1

@tomruzicka

Posted

@sirriah Ahoj Terko,

děkuji za přivítání a za skvělé rady! ☺

0
Ahmed Faisal 5,095

@afrussel

Posted

Good work.

  • User rem instead of px for font size is a good practise
  • After body use main tag
2

@tomruzicka

Posted

@afrussel Thanks for advice :-)

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