Fylo data storage component using Flexbox / animation on the bar

Community feedback

• 

  Shashi Lo•1,345

  @shashiloPosted over 4 years ago

  Great job with this new challenge. From a glance, it looks really good, but I did find some areas of improvement.

  • You need to set a max-width for the desktop of 1440px according to the design. I never use percentage padding because you do not have full control of it.
  • Hover state items should have a pointer cursor for better UX.
  • Remove the fixed body height on mobile. It's causing a vertical scroll when this design is small enough to fit on the first fold.
  • For me personally, I do not use any position: absolute items on my mobile screens. The reason why is because it's much harder to position something on a mobile screen than it is on a desktop screen. I always try to build my sites constructing my DOM elements mobile first.

  1

  ---

  

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted over 4 years ago

  @shashilo Thank you very much for the feedback, I will consider these changes!

  0