Mo• 130
@mo-rob-1
Posted
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 helpful
1
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
Rio Cantre• 9,710
@RioCantre
Posted
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--onerule 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 helpful
1
Hexer• 3,680
@Phalcin
Posted
Nice work man.
Marked as helpful
1
Mayur Mali• 810
@Mayurtm29
Posted
Border radius is missing. overall good work. happy coding
Marked as helpful
1
Peter Hannell• 200
@peterhannell
Posted
@Mayurtm29 oops! thank you.
1
Naveen Gumaste• 10,480
@NaveenGumaste
Posted
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 helpful
1
Please log in to post a comment
Log in with GitHub
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