Single price card

Community feedback

• 

  Bryan Li•3,530

  @Zy8712Posted 12 months ago

  Your site's desktop layout looks pretty good. Main thing I'd work on modifying is your mobile layout using your media queries. Right now when the screen size shrinks your box becomes very squished together. One other thing worth mentioning is that your background color doesn't extend the full length of the page when the screen size is small. This might be due to how you set up your background-color. Make sure its assigned to the body of your site and that the body fills the entire height of the site.

  Hope you find this feedback useful. P.S. Your SCSS looks pretty good.

  Marked as helpful

  1
• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 12 months ago

  Hello there 👋. Good job on completing the challenge !

  • The structure of your Sass looks good, nicely organized by category. I have a few style suggestions: consider setting a max-width to prevent your component from stretching too much on large screens. You can test this behavior using zoom. Additionally, use a media query for mobile responsiveness; consider adopting a mobile-first approach in future challenges. Lastly, this challenge is perfect for utilizing grid. While flexbox is great, grid might offer an even more straightforward solution.

  I hope you find it useful! 😄

  Happy coding!

  Marked as helpful

  1