Design comparison
Solution retrospective
My solution to this challenge. I'm not 100% satisfied with it but I think it's close.
Any feedback and suggestions much appreciated.
Community feedback
- @mo-rob-1Posted almost 3 years ago
Good job dude, looks great! Also noticed in your code that you used the BEM technique and CSS custom properties.
Keep up the good work :)
Marked as helpful1 - @RioCantrePosted almost 3 years ago
Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…
- Adjust the border radius of the buttons into
border-radius: 30px;
- Add
border-bottom-left-radius: 10px;
andborder-top-left-radius: 10px;
in.col--one
rule set - Add
border-bottom-right-radius: 10px;
andborder-top-right-radius: 10px;
in.col--three
- Apply the same borders in the responsive design but on different side.
Above all, Well done! Keep it up and Hope this is helpful!
Marked as helpful1 - Adjust the border radius of the buttons into
- @Mayurtm29Posted almost 3 years ago
Border radius is missing. overall good work. happy coding
Marked as helpful1 - @NaveenGumastePosted almost 3 years ago
Hay ! Peter Good Job on challenge
-> Check my article on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding 👍!
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