CSS Grid , Media query

Community feedback

• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hay ! Good Job Abdellah

  These below mentioned tricks will help you remove any Accessibility Issues

  -> -> Always use h1 first and then h2, h3 and so on

  Keep up the good work!

  Marked as helpful

  2
• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted over 2 years ago

  Hello @abdellahelaajjouri ,

  You can try to wrap the whole component in <div> and add overflow: hidden; border-radius: ;, on the <div> not on the <main>.

  hopefully this feedback helps

  Marked as helpful

  1
• 

  Mortey Kafui•380

  @MorteyKafuiPosted over 2 years ago

  Hi Abdellah, this is a nice project. Good work done!!!

  In order to center the entire card in on a large screen thus, desktop screens you can use the code below on the parent container.

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

  Marked as helpful

  1
• 

  Mahmoud Salah•160

  @Mahmoud2227Posted over 2 years ago

  Hey, Congratulations on completing the challenge.

  I just saw your code and I think that you were trying to change the border-radius of your <main> element and it works! but it's taking the full width of the screen so try to change its background-color and you will notice what I'm talking about.

  in this case, if I were you , I would give it max-width instead of using padding.

  Marked as helpful

  1

  ---

  

  Abdellah el aajjouri•190

  @abdellahelaajjouriPosted over 2 years ago

  @Mahmoud2227 chokrane akhi

  1