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

Fylo data storage white bootstrap & Sass

AngelARVM 175

@AngelARVM

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


Any feedback is welcomed!

Community feedback

@Amir-Jacobs

Posted

I love your usage of variables in CSS. I also like that you've used prefixes where needed.

Some things I'd do differently are:

  • Change width: x% to width: 100%; max-width: x%, as this helps with responsive design.
  • I'd limit the amount of classes I put on the html elements, because it becomes harder to read otherwise (though I understand that with bootstrap it's inevitable to add classes). I'd look into the BEM method.
  • I noticed you need to use margin-top: -340px; margin-right: -340px; on a class, which points to an issue in a part of the CSS.

Overall it looks clean and structured. :)

2

AngelARVM 175

@AngelARVM

Posted

Hi Amir, I really appreciate your comments.

I had take a look into the BEM method and i'll try to apply it. And I promise I tried to use absolute positions with the left storage speech bubble, but somehow a lot of things broke. I think I should go over these concepts.

Anyway thank you so much :)

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