Single price grid component using Flexbox

Community feedback

• 

  Rajaul Islam Ratul•1,250

  @ratul0407Posted 12 months ago

  @frankgomezdev congrats on completing this challenge🎉🎉

  You've done a really great job. There's not a lot to complain about👍. But,

  • the padding you've set to .container of 2rem is actually shrinking your container. You can just remove it and it looks more like the design.
  • It's such a small challenge and there wasn't a lot of space to add landmarks but you've done enough. The footer you've added is good but you've commented out the .attribute😅. I know this mistake 'cause I've also done a couple of times. Uncomment it and remove the .attribute div as you are using a footer
  • And the last nick pick that I can say, and it's completely my personal opinion that there are too many div element on the page. You can consider making the top box a article.

  I hope you found this helpful👍👍

  Happy coding and have a very nice day😄

  Marked as helpful

  0

  ---

  

  P

  Franklin Gomez•180

  @frankgomezdevPosted 12 months ago

  @ratul0407 Thank you so much for the feedback. Very helpful and I will implement these tweaks, have a nice day!

  0