Mariem Ehab• 335
@MaryEhb
Posted
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 helpful
0
sriram• 120
@sriramshiyam
Posted
@MaryEhb thanks you for the review, and thanks for the minor details you mentioned 🤗
0