Social Media Dashboard Challenge using HTML, CSS, Bootstrap, JQuery

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  Awesome work, Ricky, and congrats on submitting your first solution! I'm liking the font choice 😂 Your CSS naming is fine. You could look into something like BEM if you want a more structured approach to naming.

  The only thing I'd recommend is to try removing some div levels. There are a lot of nested divs in your HTML code. But then again I guess this is mostly a by-product of using Bootstrap.

  Keep up the great work and I hope you enjoyed the challenge!

  2

  ---

  

  Ricky•150

  @voyage11Posted over 4 years ago

  @mattstuddert Hi Matt, Thanks for the feedbacks! I will be looking into BEM and see how I can improve the CSS naming. I shall see if I am able to reduce the div level too. Thanks! :)

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  @voyage11 you're welcome! If you have any questions about BEM please feel free to message me.

  1

  

  Ricky•150

  @voyage11Posted over 4 years ago

  @mattstuddert Sure. Thanks in advance!

  0
• 

  Gerben Dol•3,115

  @GerbenDolPosted over 4 years ago

  Hey Ricky! Your solution is looking great! 😁 I like how you switch on the toggle text depending on the mode, that's a nice touch. 😏

  I'd love to see you apply a border radius to all the cards! I think it would make everything look just a little bit smoother. 👍

  1

  ---

  

  Ricky•150

  @voyage11Posted over 4 years ago

  @GerbenDol Thanks for the compliment! Still need to work hard to improve my codes.

  0