Responsive Time-tracking Dashboard

Community feedback

• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  Hey, great job! solution matches perfectly with the design.

  Just a few suggestions; for html, links are not good for this case as they are used for navigation to a different page, while buttons are perfect for use here. For img with empty alt, it will be great to add aria-hidden="true";, so screen readers ignore it. And hey add some color change for an active daily weekly or monthly stats, button to make it standout.

  Happy coding :)

  Marked as helpful

  1

  ---

  

  Corvida Raven•680

  @SheGeeksPosted almost 3 years ago

  @techanthere Will try these adjustments including the color change on active buttons. Nice catch on the missing alt for the ellipsis icon and appreciate the suggestion for accessibility! Will have to save that one for future projects.

  1

  

  Corvida Raven•680

  @SheGeeksPosted almost 3 years ago

  @techanthere Just a quick update that I made these changes and, again, really appreciate your suggestions for the buttons.

  1

  

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  @SheGeeks thank you, I am glad to help.

  0