Base Apparel Comming - Usin HTML, Sass (Flexbox & Grid), JS
Design comparison
Solution retrospective
Hello everyone, I had some if the content always had to cover the entire width of the screen π€ ,but that distorted all the content, so I gave it a maximum and centered width so that when the screen is bigger it will always look good and to the center. Comments are welcome.π
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello there, Kevin Morales! π
Good effort on this challenge! π Thinking about how to make this challenge responsive for bigger screens is something that stumped me a bit, too, when I worked on my solution! π
I'd like to suggest,
- Labeling the input element in some way to identify it and improve the accessibility of the page (and clear up that error on your solution report).
- Check the responsiveness of the page from around 1300px to 1000px (at that point, a horizontal scroll bar appears along the bottom of the screen and not all of the content of the page can be seen without scrolling left/right).
Keep coding (and happy coding, too)! π
1@kefer16Posted over 3 years ago@ApplePieGiraffe Hello, thank you very much.π It is an honor for me that you advise me on my challenge, as it guides me from the animations that you implemented in your solution.π
- The accessibility error in my HTML, I corrected, I didn't know that attribute (aria-label), so I could investigate its use.
- As for the horizontal bar error, I modified the image container to (overflow: hidden) so that it doesn't exceed the size of the image in terms of its container (I don't think it's the effective solution, but it shows less)
Β‘Happy Coding π!
1@ApplePieGiraffePosted over 3 years ago@kefer16
Good work on the fixes, Kevin! π
I actually find myself using
overflow-x: hidden
quite a bit here and there to get rid of any unwanted horizontal scroll bars in things I build. I just try to make sure that I take care of any point where scrolling will actually be wanted (e.g., in some cases, I want there to be vertical scrolling but no horizontal scrolling, henceoverflow-x
notoverflow
). But it's all up to the scenario and what works best! πKeep it up! π
Oh, and hey, if you found these tips helpful, an upvote would be appreciated! ;)
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord