3 Column Layout Newbie Challenge (mobile & desktop view only)
Design comparison
Solution retrospective
This is my first time completing a frontend mentor challenge. I want to be a professional frontend web developer in the future. Any feedback/tips are welcome.
Community feedback
- @b4n1y4Posted over 3 years ago
Hey Haylie, good job on your first challenge... A few things I would like to tell you are -
-
Changing the div with container class to a main, would resolve your accessibility issue. It occurs since all your containers are divs and technologies like screen readers require a more semantic approach in web pages. So, they definitely need a main section that can be focused... It will change nothing in your design. just remember to also change the style sheet accordingly.
-
For Learn More buttons, you could try border-radius: 50px;
Try these out, and tell me if it works... All the best 👍 Keep coding ✌
Marked as helpful0@b4n1y4Posted over 3 years ago@hayliegrayson You should generate the report again if you have made these changes...
0 -
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