Responsive four card feature section master

Community feedback

• 

  Dust98•130

  @Dust1100010Posted over 2 years ago

  Hey dude, congrats for the challenge ✌.
  to center the container use this: height: 900px; width: min(1000px, 100%);

  and in the "cards" class just need margin-top: 30px

  try to use diferents font sizes and colors to make tha page more similar like the original desing. Good Jod 😉

  0

  ---

  

  HShirzad•170

  @Haseebullah555Posted over 2 years ago

  @Dust1100010 thanks bro!!!

  0
• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should use <main id="container"> instead of <div id="container">.
  • Go down orderly when you are using the headings h1 down to h2 down to h3 and so on and not leave them empty.

  This should fix most of your accessibility issues

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  0