Fylo data storage component - SCSS

Community feedback

• 

  Marcos Travaglini•4,840

  @BlackpachamamePosted 10 months ago

  I could only rate your solution as perfect 😎

  It looks super correct and the animations give it an extra plus.

  I can only add:

  • You defined the same thing twice in the body, height: 100vh and height: 100dvh, technically they are not the same, but they are overwritten because the property is the same (CSS will apply the one below)
  • You could set the alt attributes as empty, for accessibility reasons, not leaving just alt but instead setting alt="". This is to improve accessibility, as in this case the images are only decorative, the description is left empty. More info
  • You could remove comments that are not useful in your HTML

  Marked as helpful

  2