Responsive Four Cards

Community feedback

• 

  Alaa Mekibes•1,540

  @alaa-mekibesPosted 3 months ago

  Hi @RanndaLL0, great job 🎉! Just a few points to fix:

  • Center your layout using Flexbox or Grid. You can use the following examples:

  1. For Grid:

  display: grid;
  place-items: center;
  min-height: 100vh;
  

  2. For Flexbox:

  display: flex; 
  justify-content: center; 
  align-items: center; 
  min-height: 100vh;
  

  Note: min-height: 100vh; ensures the layout takes up the full screen height.

  • Use CSS variables to improve maintainability, like this:

  :root {
      --bg-color: hsl(210, 46%, 95%);
      /* Add your other colors here */
  }
  
  body {
      background-color: var(--bg-color);
      /* Other properties */
  }
  

  • Change the paragraph color to match the original design and add some margin-bottom for better spacing.

  • I don't why you add style1.css style2.css index 1 index 2 this is confuse. I think you should give each project a repo.

  Keep refining, it’s looking great so far!

  1