Time Tracking Dashboard

Community feedback

• 

  Brayden Williams•260

  @redstar504Posted 12 months ago

  Nice work Chris!

  My only advice is that you may want to avoid getting in the habit of applying too many classes, and using too many DIVs, or other general purpose elements to represent your content. It makes the HTML less readable, and harder to maintain.

  It's better practice to take advantage of the vast amount of tags HTML has available and to represent each type of content on the page with them semantically. That way you can simply target the elements using selectors rather than depending on a lot of verbose class tags. I personally try to only add classes or IDs when it's absolutely necessary.

  I whipped this example up very quickly to give you an idea of how little markup you could really get away with. Take note of how much more readable it makes the HTML. I did not spend a lot of time refactoring the CSS, so it may not function perfectly. I tested using Chrome on Ubuntu.

  https://redstar504.github.io/fm-time-tracking-dashboard/

  Marked as helpful

  1

  ---

  

  P

  Chris•770

  @ccccchrizPosted 12 months ago

  Thanks @redstar504!

  I have started to implement your suggestions, it's already easier to work with and there is still some stuff that is absolutely unnecessary.I will definitely use child selectors more often.

  Thanks again for taking time to help me out.

  0