CSS Grid , Media query

Community feedback

• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 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!

  20

  Marked as helpful
• 

  PhoenixDev22•16,830

  @PhoenixDev22

  Posted over 3 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

  10

  Marked as helpful
• 

  Mortey Kafui•380

  @MorteyKafui

  Posted over 3 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;
  

  10

  Marked as helpful
• 

  Mahmoud Salah•160

  @Mahmoud2227

  Posted over 3 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.

  11

  Marked as helpful