@francisldn
Posted
Hi @ohermans1 nice job! Everything works smoothly and it is fully responsive. Well done! The only comment I have is regarding the search bar. I personally think that it's not aesthetically pleasing to display the search bar if it's empty, so for my build, I chose to hide it if it's blank. But this is purely down to personal preference. You can take a look at mine here: https://www.frontendmentor.io/solutions/job-listing-search-with-nextjs-tailwind-and-typescript-BJci7g6I5 Feel free to comment on my solution too.
Marked as helpful
@ohermans1
Posted
Hi @francisldn - thanks so much for your feedback.
I have taken it on board, and have added an opacity toggle to the search bar once it is no longer empty (I went with opacity so that the listings didn't jump around when the search bar appears - this would be my feedback for yours (everything else looks great).)
I also added some additional filter functionality. If you click one of the already selected filters on a listing. It should remove the target filter.
I would love if you could take another look and my preview site and let me know what you think.
Ollie