-
Notifications
You must be signed in to change notification settings - Fork 472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/add reset default prop selected filters component #1725
Feat/add reset default prop selected filters component #1725
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mohdashraf010897 There are some formatting changes. You can sync your branch with next
branch and then fix the changes.
and tested list components
@@ -30,6 +31,7 @@ const SelectedFilters = { | |||
selectedValues: this.selectedValues, | |||
clearValues: this.clearValues, | |||
setValue: this.setValue, | |||
resetValuesToDefault: this.resetValuesToDefault, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mohdashraf010897 You would have to update the documentation for it,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mohdashraf010897 Looks good now 👍
Description:
Added
resetToDefault
prop in SelectedFilters component, WhenresetToDefault
is set to true, then it would set the filter value to thedefaultValue
.The default value for
resetToDefault
would be false.Libraries Affected 👍
Feature Request PR
Notio Card
web
Demo with resetToDefault prop
Demo w/o resetToDefault prop
MultiDropdownRange
Demo with resetToDefault propvue
Demo with resetToDefault prop
Demo w/o resetToDefault prop
Range Input
Demo with resetToDefault propLoom Demo:
Latest
ImplementationLoom Demo
Docs PR