@elaineleung
Posted
Hi Prince Roy, well done completing this! 🙂 I checked it out just now, and it looks like your grid is off to the left side and doesn't look like your screenshot, so you might want to see what happened there. I like your use of insertAdjacentHTML
and I also found it insightful how you wrote some of your functions.
Two quick suggestions I have are, you can use cursor: pointer
for the clickable timeframe labels, and also I think the "Last week" label in the cards actually should be changeable to reflect the time frame selected (e.g., "Yesterday" for the "daily" data). Great work here once again!
Marked as helpful
@iprinceroyy
Posted
Thanks for your feedback @elaineleung. I've updated it.