blog-preview-card

Community feedback

• 

  P

  Arcloan•680

  @ArcloanPosted 3 months ago

  Hi, the solution looks really great :) For the responsiveness of web pages you should stop using fixed width and height and start using, max-width to limit the width of a container, and possibly don't set a value for the height of a component. If you start thinking in max-width term you'll find it much more easy to build responsive pages.

  E.g. You could have used on the container max-width : 25rem and margin-inline : auto. (You used grid on the body and place-items to center it. It is equally valid)

  Anyway I recommend the youtube channel of Kevin Powell. He has a lot of amazing videos. You could learn a lot.

  Marked as helpful

  1

  ---

  

  volti42•50

  @volti42Posted 3 months ago

  @Arcloan Thank you for your response. Do you have any other suggestions for improvements in any part of the code structure?

  0

  

  P

  Arcloan•680

  @ArcloanPosted 3 months ago

  @volti42 Not very much since the project is small. Learn first flexbox and then grid, they will help you tremendously in yur journey. Don't jump straight into a framework e.g. tailwind (I personally love tailwind but it is better to stick with vanilla css in the beginning so that you have a much better basis to learn new thing) and remember to use HTML only to structure your content not to give the content a specific decoration. Keep that in mind. You used an h5 element jumping many heading levels. Also don't use id selector in css and never use important in a css rule. Later in your journey you'll understand better why when you'll find yourself in a battle for specificity.

  0