Pricing Component With Toggle.

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Hey Saurav, what mistakes do you feel that you've made while building this project? Here are a few pointers after taking a look at your code:

  • You're not loading in the 700 weight that you need of Montserrat to display the content correctly. This is making your content appear thinner vs the design.
  • You're using multiple h1 elements on the page. I'd always recommend only using a single h1, which is your main page heading.
  • You've got h1, h3 and h4 elements, but not h2. Avoid skipping heading levels unnecessarily as it can lead to accessibility issues with the content hierarchy.

  I hope these tips help. Keep up the great work!

  0

  ---

  

  Saurav Chamoli•370

  @sauravchamoli17Posted almost 5 years ago

  @mattstuddert I've fixed the multiple h1 present in the project but I am not able to fix the font weight of the content and the background pattern is also not visible. Can you help me out with this?

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  @sauravchamoli17 to load in font-weight: 700 from Google Fonts you'll this link tag instead of the one you've got: <link href="https://fonts.googleapis.com/css?family=Montserrat:700&display=swap" rel="stylesheet">. I'd recommend having a deeper look at Google Fonts to see how you can customise the fonts that you load in to add different weights.

  For the background patterns, you've just got the wrong path for the images. You need to change url(/images/bg-top.svg), url(/images/bg-bottom.svg) to url(../images/bg-top.svg), url(../images/bg-bottom.svg).

  0

  

  Saurav Chamoli•370

  @sauravchamoli17Posted almost 5 years ago

  @mattstuddert Thanks.

  0