RESPONSIVE LANDING PAGE USING CSS FLEX BOX

Community feedback

• 

  HaDo Dev•310

  @hadodevPosted 2 months ago

  Hi @Grahamocean!

  • Nice pulse animation.
  • Try to acheive better the space between the elements.
  • Also pay attention to the font weight of the author. It's must be higher.
  • For the Learning label you might give it a better style. Here is an example:

  .label {
          display: block;
          width: 12ch;
          padding: .5rem;
  
          font-weight: var(--fw-bold);
          font-size: .875rem;
          text-align: center;
  
          background-color: var(--clr-yellow);
          border-radius: .25rem;
          padding-inline: .75rem;
  }
  

  The custom properties used at :root are the following:

  --fw-bold: 800;
  --clr-yellow: hsl(47, 88%, 63%);
  

  • For a better accesibility, use rem unit instead of px for fonts, border-radius, padding, etc...

  Good job! Keep going :-)

  Marked as helpful

  0

  ---

  

  Grahamocean•50

  @GrahamoceanPosted 28 days ago

  @hadodev I appreciate your review. I'll work on the changes!

  0