Make CustomClipper extend Listenable #55995
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.
Description
Currently CustomClipper does not allow outside libraries to listen to the repaint listenable provided to its constructor.
This PR fixes that by making CustomClipper extend Listenable just like CustomPainter.
Related Issues
#55992
Tests
I added the following tests:
clip_test.dart
/CustomClipper reclips when notified
Checklist
///
).flutter analyze --flutter-repo
) does not report any problems on my PR.Breaking Change
Because this change adds methods, it could break code that implements CustomClipper instead of extending it.
Did any tests fail when you ran them?