Responsive Solution using flex

Please log in to post a comment
Log in with GitHubCommunity feedback
- @khatri2002
The design looks good! Just a few corrections and suggestions:
- Border Radius for Smaller Width Devices:
In smaller width devices (where the flex container gets wrapped), the upper
sub-container
should only have a bottom border radius (as per the design reference). Theborder-top-right-radius
andborder-top-left-radius
should be removed for these devices. You can achieve this using a media query.- HTML Structure:
- Use the
<main>
tag to semantically define the main content of the page. - Add an
<h1>
tag as it is essential for accessibility, SEO, and defining the primary heading of the page content.
Implementing these changes will align the design and structure with best practices.
Happy coding!
Marked as helpful
Join 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