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

Completed It! But With Sass Or Scss.

@vedantlamba

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


I Used Css-Preprocessor Called Sass! Done It Using Pure Html/CSS(scss). Enjoyed It! Thankyou For The Challenge.

Community feedback

@Agnik7

Posted

Hi,

Hope you're doing well.

I went through your solution to this challenge in which you have done a great job. However, I felt there are some things that I learnt while doing these challenges, which might help you as well.

  1. You are providing a max-width to the .card element. I feel that you have already made it responsive, and thus all those media queries are unnecessary.
  2. Try to avoid using position: absolute , since it might result in overflow on some screen sizes. You can try using flexbox or grid to achieve the same. To learn different types to center a div, click here.
  3. Replace the <h5></h5> tag with the paragraph <p></p> tag, it will achieve the same result, at the same time, fixing the accessibility issue.
<p>$59.99/year</p>

Hope this comment helps you. If you feel that I have said something incorrect, please do correct me without any hesitation.

Have a nice day!!

Marked as helpful

0

@vedantlamba

Posted

@Agnik7 Thankyou So Much For Guiding Me, I'll Definitely Re-code All The Things You Said! Once Again ThankYou For Your Valuable Time!

1

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