@danielmrz-dev
Posted
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!
@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?