Submitted over 3 years ago
Responsive Base apparel using CSS, Flexbox, CSS Grid, and JS
@JacksonJ97
Design comparison
SolutionDesign
Solution retrospective
Any feedback is greatly appreciated! :)
Community feedback
- @ChamuMutezvaPosted over 3 years ago
- input elements must have a label to assist users who rely on assistive technology on what they are expected to do. The label must have a clear message. The option you can use here is to have a label that is visually hidden from sighted user.
- another important aspect of responsive design is responsive images - where you can use techniques such as srcset or picture element. The method you used as shown below is not best practice - it is considered expensive to use and can slow down the loading of the site as every image in the html has to be downloaded
<div class="illustration-container"> <img src="images/hero-mobile.jpg" alt="" class="mobile-hero-image" /> <img src="images/hero-desktop.jpg" alt="" class="desktop-hero-image" /> </div>
- Looks great on mobile.
Good lucky
Marked as helpful1
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