Skip to content
Submitted about 3 years ago

Order summary component

David Kalany•50
@AmazingCukr
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi all,

this is my 2nd attempt to make this card. I would appreciate any help/criticism to make my code better.

Thanks for all comments!

Community feedback

  • nmorajda•945
    @nmorajdaPosted about 3 years ago

    Look fine on the desktop, but there are some small bugs in the code:

    <div class="Header"> ...

    Don't use uppercase letters at the beginning of the class name, id, etc.

    <div class="text"> ...

    This div element does nothing.

    Why not:

    <p class="text"> ....

    ?

    It is also not responsive and from a width of 400px a horizontal scroll bar starts to appear.

    You also have three errors in the report above.

    0
    David Kalany•50
    @AmazingCukrPosted about 3 years ago

    @nmorajda I forgot about not using uppercase haha. Also p class looks better than making another useless div. Im very new to HTML and CSS so, Im not really sure how i can make it responsive tho :(

    0
    nmorajda•945
    @nmorajdaPosted about 3 years ago

    @AmazingCukr Key words: media query, responsive web design.

    Happy codding.

    0

Please log in to post a comment

Log in with GitHub

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