Stats Preview Card

Community feedback

• 

  Zeena•175

  @kushyzeePosted over 3 years ago

  I have gone through your code and I have the following suggestions

  1. Remove the style tag from your css file, it's a html tag and has no business in css.

  2. Your <div class="context"> is missing a closing div tag.

  3. Consider removing the height from .context, instead you should allow the contents to determine the height (which you can manipulate by using margins and paddings. Also don't use px in the width property, use % instead so as to make the site responsive across different screen size (90% will suffice in this case).

  4. The line height on the h1 property is too much. Consider reducing it

  5. The image was supposed to be used as a background-image and not as a direct image in html file. But I guess that's what works for you

  There are a bunch of other things you could have done differently, I can't exactly get into it right now without changing the entire code

  0