"Filter by" option to filter by the value of the cell

[mnmt7]

Solves #1155

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
rowy-os	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 25, 2023 5:29am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
rowy-typedoc	⬜️ Ignored (Inspect)			May 25, 2023 5:29am

[vercel]

@mnmt7 is attempting to deploy a commit to the Rowy Team on Vercel.

A member of the Team first needs to authorize it.

[harinij]

Hi @mnmt7 - Thank you for this PR. I tested this and it works well. Suggest a few minor changes.

1. The older "Filter value" option needs to be removed. That option used to open the filter popup modal and then user had to click Filter button again for the filter to show up. This newer "Filter by" option is the expected behavior to quickly filter by a value in a table. So the older "Filter value" be deprecated to avoid double options.

2. I also noticed that this "Filter by" option is disabled on fields which are spread out versions of a JSON - however manual filter on this field works well. It would be good if "Filter by" is enabled in this field as well as double checking for other field types. Quick video explainer.

[mnmt7]

@harinij, I have implemented the changes. Please review when possible.

[htuerker]

Hey, @mnmt7! Thanks for your contribution! LGTM!