Page made with Flexbox

Community feedback

• 

  Talika Bajaj•650

  @Talika-BajajPosted 8 months ago

  Hi @MichaElL0,

  Your project looks great.

  I'd like to suggest a way to make it even better:

  • Giving margins when using Flexbox Layout isn't a good practice. Instead, you should try and use the gap property that will space your text evenly inside a container.

  .container {
       display: flex;
      align-items: center;
      flex-direction: column;
      gap: 20px;
  } 
  

  • Also, I suggest using the:root pseudo Class Selector to specify colors and then use them later in your code. For Example,

  :root {
      --Green: hsl(75, 94%, 57%);
      --White: hsl(0, 0%, 100%);
      --Grey: hsl(0, 0%, 20%);
      --Dark_Grey: hsl(0, 0%, 12%);
      --Off_Black: hsl(0, 0%, 8%);
  }
  

  body {
      background-color: var(--Off_Black);
  } 
  

  It is really helpful and keeps you hassle free from copying colors again and again. To know more, you can read this article - :root pseudo-Class Selector

  Good Luck!

  Marked as helpful

  0