Html and Css

Community feedback

• 

  dewslyse•3,025

  @dewslysePosted about 3 years ago

  Hello @ChitThetAung! Congrats on your submission. I believe you have used the grid layout quite well and as such your page is responsive across devices. However, there are some things to note:

  • You should set a max-width (eg. 1140px) on your .container to prevent content from spanning the full width of the page on large devices.
  • You could centre the testimonial grid to the page by adding the following to the body

  body {
     display: flex;
     justify-content: center;
     align-items: center;
  }
  

  • The main content of your page should be wrapped with the main tag.(e.g.: replace <div class="container"> with <main class="container"> This would help resolve accessibility issues regarding landmarks.

  Overall, nice job.

  Marked as helpful

  0

  ---

  

  Chit_Thet_Aung•195

  @ChitThetAungPosted about 3 years ago

  @dewslyse Wow Thanks bro for comment me. Your suggestion is really helpful for me.

  0
• 

  Chit_Thet_Aung•195

  @ChitThetAungPosted about 3 years ago

  wow ! So many thanks u bro.

  0