Employed grid and flex

Community feedback

• 

  P

  The Burrito Doggie•1,260

  @BurritoDoggie

  Posted over 3 years ago

  Hello there!

  Amazing job here! You did really well on this challenge! One thing you could add here is a div on the image at the top. Then you can add border-radius to the top corners. Great effort!

  Keep Coding!

  :)

  (If you found this helpful you can let me know by marking this helpful or by giving this a thumbs up!)

  24

  Marked as helpful
• 

  P

  Jessica•880

  @perezjprz19

  Posted over 3 years ago

  Hey, you're almost there. A few things to work on and you should be a little closer to your goal.

  If you wrap the div with the class .card inside <main> tags, that will get rid of your accessibility errors in the report.

  The <div> with the class .container can be replaced by <button>. It should look something like this <button>Button Text goes here </button>. The reason for this is because divs have no semantic meaning. You can make them look like things and mimic the same actions in some cases, but they are still not the real thing.

  Happy coding!!

  11

  Marked as helpful