First try with Front-End Mentor building an File Storage Component

Community feedback

• 

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted over 4 years ago

  Hello Benjamin,

  Very nice first solution! You did a really good job! I applied these properties to the background to add and position the background image:

  background-image: url("images/bg-desktop.svg"); background-repeat: no-repeat; background-size: cover; background-position-y: 50vh;

  Also, you should consider to modify the breakpoint where the boxes change the flex-direction, 600px is too late and the boxes overflow the body.

  And the last thing, add cursor: pointer; and hover states for all clickable elements.

  Happy coding! Keep going!

  2

  ---

  

  Benjamin Sabardu•90

  @bsabarduPosted over 4 years ago

  @ovidiuantonio thanks for this feedback, i will try these and update the solution !

  0
• 

  Abdulla Bashiroglu•520

  @bashirogluPosted over 4 years ago

  Good job Benjamin, there is a small problem with responsiveness in sizes of 600 800. Maybe an early mobile layout would solve the problem. Good luck.

  1

  ---

  

  Benjamin Sabardu•90

  @bsabarduPosted over 4 years ago

  Thanks for your feedback. Indeed i will improve de responsive breakpoint :)

  0