CSS Grid CSS Flexbox

Community feedback

• 

  P. Ricardo•2,370

  @pRicard0Posted about 1 year ago

  Some HTML tips

  • Instead of using the tag <span> with the text "insights", you can use the tag <em>. The <em> tag is used to define emphasized text. The content inside is typically displayed in italic. A screen reader will pronounce the words in <em> with an emphasis, using verbal stress.
  • You can use <main> instead of <article>

  CSS tips

  • Your project is not responsive on larger screens. You should add this...

  .container {
      ~~width: 144rem;~~ ( this is an error )
  
      width: 100%;
      min-height: 100vh;
      display: flex;
      flex-direction: column;
      justify-content: center;
      align-items: center;
  }
  

  • margin in the article isn't necessary

  Marked as helpful

  0