Submitted almost 3 years ago
Time tracking dashboard challenge using typescript
@sriramshiyam
Design comparison
SolutionDesign
Community feedback
- @MaryEhbPosted almost 3 years ago
The site looks pretty the same as the design and also you are the only one I reviewed who added the hover style to the three dots icons so great job 👏👏 I just have a few remarks:
- There are more than one element that has the same id 'divsm' and that is not right because the ids are unique and can't be repeated for several elements. You can use classes instead
- in the same elements as the previous point I noticed that the number of hours change as the user choice but the text (Last week) stay the same where it should also change between (last day - last week - last month)
Marked as helpful0@sriramshiyamPosted almost 3 years ago@MaryEhb thanks you for the review, and thanks for the minor details you mentioned 🤗
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord