Single price grid component

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 10 months ago

  Hello @Matt-LaRochelle!

  The organization of your code looks pretty good to me!

  I have just one suggestion:

  • Remove this line from your code:

  main {
      overflow-y: scroll;
  }
  

  This is causing a vertical scrollbar to appear sometimes. I don't see any reason for a scrollbar on this project, since the whole content is visible both on mobile and desktop versions.

  I hope it helps!

  0

  ---

  

  P

  Matt•50

  @Matt-LaRochellePosted 10 months ago

  @danielmrz-dev

  Thank you for your feedback!

  The reason I included a vertical scrollbar is because if someone is on a phone and turns the screen sideways, you will need a scroll in order to see the whole project. I don't know if it only has to be viewable at 375px and 1440px as it says those are the two design widths for this project? I always want the project to be viewable on any screen size so that's the requirement I set for myself... Maybe I am trying to cover too many edge cases?

  0