Fylo Data Storage Component With SCSS and JS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hi there, Luiz Claudio Lopes! 👋

  Nice job on this challenge! 👏 Your solution looks good and is responsive! 😀 And I love that you decided to make the progress bar an actual slider (that's something I haven't seen before)! 😆 Also, you remembered to add hover states to the options in the box to the left (something often overlooked in this challenge, LOL)! 👏

  Keep coding (and happy coding to you, too)! 😁

  1

  ---

  

  Luiz Claudio Lopes•505

  @lc-dev90Posted over 3 years ago

  @ApplePieGiraffe Thank you ApplePieGiraffe!

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  look at your layout in the browser at 800px wide. I would try to make it so both items are the same width on the page during that transition between mobile and desktop

  1

  ---

  

  Luiz Claudio Lopes•505

  @lc-dev90Posted over 3 years ago

  @palgramming Thank you again Patrick! I have made some adjustments

  0

  

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  @lc-dev90 Looks better and congrats on styling the range slider for this project. you could have just styled div's to create a fixed scale also

  1

  

  Luiz Claudio Lopes•505

  @lc-dev90Posted over 3 years ago

  @palgramming omg how i did not think about it! Thanks again my friend!!!

  0

  

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Good job, since you chose to implement the slider as form control, you could wrap inside a form element instead of div becasue it's more sementic.

  Keep coding :)

  1

  

  Luiz Claudio Lopes•505

  @lc-dev90Posted over 3 years ago

  @MojtabaMosavi Thank you!

  0