-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: animationAnimation APIsAnimation APIsc: API breakBackwards-incompatible API changesBackwards-incompatible API changesc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterframeworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.
Description
The class AnimatedBuilder should have its name replaced. As it is not related to animation in reality.
For example, the following is possible :
final valueNotifier = ValueNotifier(0);
AnimatedBuilder(
animation: valueNotifier,
builder: (context, _) {
return Text(valueNotifier.value.toString());
},
)Same goes for ScrollController, ...
I suggest to rename it to ListenableBuilder as it makes more sense.
davystrong, tomgilder, esDotDev, icnahom, its-me-mahmud and 22 more
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work lista: animationAnimation APIsAnimation APIsc: API breakBackwards-incompatible API changesBackwards-incompatible API changesc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterframeworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.