Add beforeChange handler and loading class/state #359
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I wanted to add a loading state to this, as I'm working on a project where we save the response immediately to the server, and only want to move the check if that save is successful.
Looked at the open issues and saw the before change request, so I added that too since we can use that for this purpose as well.
#314
beforeChange is an Input that accepts an Observable that returns a boolean. If the boolean is true, the click event continues as normal. If no observable exists, the code continues as normal.
I updated the class to display it as flex and align/justify the items to center. This is more precise centering with icons that the example and flex has like 95%+ compatibility.
https://caniuse.com/#feat=flexbox
Also added some examples.