Completed It! But With Sass Or Scss.

Community feedback

• 

  Agnik Bakshi•480

  @Agnik7Posted over 1 year ago

  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

  ---

  

  Vedant Lamba•30

  @vedantlambaPosted over 1 year ago

  @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