Hello! Great job! It seems to me that the duration of the hover effect is too short to perceive, I usually set it to 300-400. By the way, tailwind has great built-in animations, for example: https://tailwindcss.com/docs/animation#bounce The background on the cross icon in the filter does not change on hover as on the layout. I also think it’s worth setting a flex-wrap on the filter container so that the Clear button is at the bottom with a large number of filters and a small screen size. All this is purely my subjective opinion! I hope my comment was helpful!
Marked as helpful
@AntoineC-dev
Posted
@MaxTarasevich Actualy you are right i forgot the hover effect on the remove btn.
I also think the wrapping is a good idea and not a problem since the filter bar is hidden in the screenshot 😜
As you made me go back to the code i also added a check on the addFliter method if is it already in the filters Array. I also disable the tag in the jobs cards so you don't try to do t anyway.
Nice review !
Thank you bud
