@elaineleung
Posted
Hi Remtaine, about not having the background hover effect when hovering on ellipsis, I think I might have had that in my earlier versions of this challenge, but eventually I felt it was just poor user experience and removed it. Anyway, maybe you can try using a mouseenter and mouseleave event listener that can remove the effect somehow.
By the way, this was well done! I think you just need some more padding to make the cards a bit bigger and also the corners a bit rounder, and that should get the component looking closer to the original. One last comment I have is, the "Last Week" label is currently also used for the "daily" and "monthly" timeframes; you may want to change that to the appropriate labels, as in "yesterday" and "last month".
Hope some of this helps you!
Marked as helpful
@remtaine
Posted
@elaineleung Thank you! I agree that it doesn't feel like good UX, so maybe I'll get rid of that in custom iterations. I'll make the necessary changes to the size and rounding, as well as changing the labels.
