-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
a: qualityA truly polished experienceA truly polished experiencec: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages team
Description
Is there an existing issue for this?
- I have searched the existing issues
- I have read the guide to filing a bug
Use case
MaterialApp(themeAnimationStyle: AnimationStyle.noAnimation)) should remove AnimatedTheme from widget tree.
When doing so was breaking a lot of tests but turns out it AnimatedTheme needed to be replaced with just Theme when using AnimationStyle.noAnimation.
Proposal
Replace AnimatedTheme with just Theme when using AnimationStyle.noAnimation in the MaterialApp.themeAnimationStyle
Metadata
Metadata
Assignees
Labels
a: qualityA truly polished experienceA truly polished experiencec: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages team
Type
Projects
Status
Done (PR merged)