Responsive card component using both flexbox and css grid

Community feedback

• 

  Md Raihan Alam•520

  @Md-Raihan-Alam

  Posted about 3 years ago

  First of all, there is nothing wrong with having a big CSS file. Second, remove all comments and keep the necessary ones, it's a bad habit to have so many comments in one file. Third, You do not have enough knowledge about rem yet practice some more because I do not find your site responsive on my PC. Fourth, you can check this video to understand when to use CSS unit ->https://www.youtube.com/watch?v=N5wpD9Ov_To.

  As a suggestion, you can use percentages to make your content responsive. I use percentages mostly in width to make my content responsive.

  11

  Marked as helpful
• 

  Ciceron•940

  @MarcusTuliusCiceron

  Posted about 3 years ago

  Hey, happy to see a fellow french citizen here :) However I'll stick to english for the sake of everyones understanding.

  There is nothing wrong about your css file size. it is not that big. You simply can't afford to not use the css properties you need. if you want to reduce your file size you will have to increase the number of file you have. This is usually a good practice when the layout to implement become richer. Another thing you can try is learning Sass, it's very close to css but have some extra features that in some case can reduce the size of your css (loops, conditions, nesting, etc)

  Hope this will help you

  Have a good day :)

  10

  Marked as helpful
• 

  Ali Ahmed•680

  @Dany-GitHub

  Posted about 3 years ago

  I suggest to use css variables and u can expand your knowledge more by learning pre processor as sass or scss u will learn things as mixin , functions, extend, partials and more it will make your CSS short and u dont have to repeat so much stuff over and over, Happy coding ✌😊

  11