Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive layout using flex box and some customise components.

@KiriRatanak

Desktop design screenshot for the Fylo data storage component coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


For a component such as this I think I have used too much CSS and it looks a bit messy. I would appreciate some feedbacks on how to do better.

Community feedback

Roman Filenko 3,335

@rfilenko

Posted

Hi Kiriratanak, seems like a preaty good work, a few notes thought:

  • try to be consistent with values (like margins, paddings, elements sizes, for example 2px, 14px, 15px, 9px, 81.5% and so on);
  • some css properties are inherited, so no need to overwrite them;
  • try to use more semantic tags;
  • would make icons as links.

Keep practicing, Roman😉

1

@KiriRatanak

Posted

Thank you @rfilenko for the comment. I'll keep that in mind.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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