Design comparison
Solution retrospective
As a warm-up project, it's simple. The project focuses on how front-end developers can flexibly use layout
Community feedback
- @YuliaLantzbergPosted 3 months ago
Hi. It's looking great as a fit to design. But you use px in the CSS and it makes it less responsive. For example, if you shrink it to mobile width the second button's text expands on 2 lines, and cuz of that its height becomes greater than other buttons. I guess this problem would be resolved if to use rems. And it's really simple if you set in HTML element font-size to 62.5% so that all rems will be multiplies of 10. Also, as you've chose to use react framework, then it could be composed of components like a Card and Button. Hope it's somehow helpful. :-)
0@erath-risePosted 3 months agoHi, thanks for your review and feedback. I agree with you, and also It would be better if I use UI framework in React. I'll try that again :)
1
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