Responsive stats preview card component

Community feedback

• 

  P

  Alper•1,010

  @adonmez04Posted almost 2 years ago

  Hi, @EvolxTV. I checked your code. It's a good solution. Keep coding. I have some suggestions for you.

  1 - You should wrap .img-container in your section tags, because it's part of the section, not a independent area in your page (as a nested structure rule).

  2 - You don't need to use span tag for .group-content's items. You can use strong and p for them. Span is an inline element, that's why you had to br tag inside of them. Keep it simple and just use the text formatting elements for texts.

  3 - You can use your class name instead of element selectors to style the elements. Element selectors have the lowest specificity, it'll be a problem for a big project. I think you already know that, I just want to remind you. Keep it simple and just use classnames.

  4 - You can use CSS Grid for the general page layout and flexbox for 1-dimensions document flow. It'll be easy to create the layout. You used the flexbox for this page layout, it's ok. Actually, I didn't do that, I failed honestly, congratulations to you for that but CSS Grid is so easy for the page layout.

  I hope these will help you to improve your perspective. Keep coding. All the best.

  1