Skip to content
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(filter component): add parameter for apply filters button text #493

Merged
merged 1 commit into from Jun 8, 2023

Conversation

stalwartsolutions
Copy link
Contributor

Issue or RFC endorsed by the maintainers

#486

Description of the change

Added the ability to override the filter component's 'Apply filters' text using the newly-added submit section of the component's parameters.

Alternative designs

This text could be updated externally using JavaScript or similar, but this allows the users to configure it from within the application's parameters.

Possible drawbacks

Conceivably this functionality could be used to add a less descriptive or misleading description, but that would be an issue for the user themselves.

Verification process

The filter example was updated to verify this functionality locally, but updating the button description in the documentation permanently seemed to present the option as mandatory.

Release notes

Added the ability to override the filter component's 'Apply filters' button text.

@stalwartsolutions stalwartsolutions requested a review from a team as a code owner June 7, 2023 16:09
@gregtyler gregtyler merged commit 5362f1e into main Jun 8, 2023
8 checks passed
@gregtyler gregtyler deleted the feature/486 branch June 8, 2023 08:55
gregtyler pushed a commit that referenced this pull request Jun 8, 2023
# [1.8.0](v1.7.0...v1.8.0) (2023-06-08)

### Features

* **filter component:** add parameter for apply filters button text ([#493](#493)) ([5362f1e](5362f1e))
@gregtyler
Copy link
Contributor

🎉 This PR is included in version 1.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants