"Clear All" Button added to clear applied Filters

[mayukh551]

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description

Added a "Clear All" button on the filter modal that clears all applied filters on react play

Fixes #551

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[vercel]

@mayukh551 is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel.

@reactplay first needs to authorize it.

[vercel]

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

Name	Status	Preview	Updated
react-play	✅ Ready (Inspect)	Visit Preview	Oct 5, 2022 at 9:41AM (UTC)

[atapas]

@mayukh551 Two quick observations

• Can we make the Clear All a link than a button? It will bring a better UX I believe
• When I click on the clear all, why does the background screen get refreshed? There may not be a functionality impact but the parent component shouldn't have impact. Can you please check?

[mayukh551]

@joshi-kaushal
Done!

[Sachin-chaurasiya]

Few comments.

[mayukh551]

@Sachin-chaurasiya Done!

[supminn]

Hi @mayukh551
I was going through your chain of commits for this PR on implementing clearAll feature.

I tried to come up with an easier solution -- click here

Let me know your thoughts on this.

cc @Sachin-chaurasiya @Angryman18

[atapas]

Hi @Angryman18

Can you please approve it if all good?

[Angryman18]

Hi @mayukh551 I was going through your chain of commits for this PR on implementing clearAll feature.

I tried to come up with an easier solution -- click here

Let me know your thoughts on this.

cc @Sachin-chaurasiya @Angryman18

i have checked the PR as its not covering the edge case as mention #562 (comment)
so we are yet good to merge this for now i think and if you find more sorter way of handling the case apart from current changes please create a fresh issue and PR @supminn

[atapas]

The clear all filter works as expected. Thanks for adding this feature @mayukh551 . Approved.

[atapas]

@all-contributors please add @mayukh551 for Code

[allcontributors]

@atapas

@mayukh551 already contributed before to code