Social media dashboard with grid, flex, CSS & js

Community feedback

• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted over 3 years ago

  Hey @JorgeDevJ. I really love the work you have done with the site and I personally think that you did a great job! The only thing I can say is try not to use I.D's in some instances to avoid HTML issues and I'd recommend you to use classes instead in that you don't have to use them uniquely. Also you could add labels to your form elements for easy accessibility by those that may have a few visual impairments and may require assistance from a screen reader.

  Marked as helpful

  2

  ---

  

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  @tesla-ambassador I.D's in some instances to avoid HTML issues To avoid what issues?

  0

  

  Jorge Durón•130

  @koikiss-devPosted over 3 years ago

  @tesla-ambassador Hi, thanks for your answer; I really don't know if I've solved it yet but I've just made some adjustments with what I've been told.

  0
• 

  Benjo Quilario•1,810

  @benjoquilarioPosted over 3 years ago

  Well done! Jorge Durón

  The only thing I might suggest is to add a max-width to some of the sections on the page so that they don't become too wide on the content inside. I make it wide to 1800px and It's too wide.

  keep coding 🔥😊

  Marked as helpful

  1

  ---

  

  Jorge Durón•130

  @koikiss-devPosted over 3 years ago

  @benjoquilario thanks for that tip, I added that to the cards above.

  0
• 

  P

  j5•2,290

  @jmnyaregaPosted over 3 years ago

  Add some padding on the cards too.

  Marked as helpful

  1

  ---

  

  Jorge Durón•130

  @koikiss-devPosted over 3 years ago

  @jmnyarega ready, is that okay?

  0

  

  P

  j5•2,290

  @jmnyaregaPosted over 3 years ago

  @JorgeDevJ Looks better 🙂

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  The main thing I see is that the user has to click the circle in the mode slider to change modes it would be nice if the user could click any place in the slider for it to change

  Marked as helpful

  1

  ---

  

  Jorge Durón•130

  @koikiss-devPosted over 3 years ago

  @palgramming I solved it :)

  1

  

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  @JorgeDevJ Looks great, Much Better

  0