All comments

• 

  deenka• 70

  @nomatter-py

  Submitted over 2 years ago

  

  HTML using CSS Grid

  1

  2

  0

  

  Zeena• 175

  @kushyzee

  Posted over 2 years ago

  You should fix the background image and have it resembling the original design.

  • You can do that by either nesting another div in the <div class="right-section"></div> and giving it a background color of #a95dda. There after you can reduce the opacity to around 0.5 or lower. (Consider changing the hex color to rgba)

  • you can also use background-blend-mode: multiply; in the <div class="right-section"></div> to fix the color

  After this you can remove the -webkit-filter property

  You did a really good job overall 👍

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Sreekanth Sreekumaran• 10

  @sreekanthsreekumaran

  Submitted over 2 years ago

  Looking for valuable feedback, and a suggestion for the mistake at the right bottom side of the card.

  

  Stats Preview Card

  1

  1

  0

  

  Zeena• 175

  @kushyzee

  Posted over 2 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