Skip to content
Submitted almost 4 years ago

HTML and CSS

HARSH AGARWAL•10
@harshag3036
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I have majorly used the POSITION attribute for positioning various text elements. Please tell whether it is the most correct way or I can use FLOAT or ALIGNMENT for positioning.

Code

Please log in to post a comment

Log in

Community feedback

  • Dušan Lukić•1,660
    @dusanlukic404

    Hey Harsh, congrats on completing this challenge. I have some suggestions for you:

    • You should use proper font-family.
    • Make the card smaller and set the background size: contain on body background image.
    • Your attribution should be outside the card.
    • I suggest you to use flex or grid layout instead of positioning everything perfectly.
    • It is not responsive so make sure to change it.
    Marked as helpful
  • Jannie Hansen•110
    @Jawha3

    Hello harshag3036, I was actually about to start this challenge myself. I think I will be using some CSS grid or flexbox to acheive the card layout. I am far from being an expert on the subject and cannot really comment on what is best practice in this scenario - that said, I do think you could have saved yourself a headache with the positioning had you used grid or flexbox.

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