Add NoAnimationPageTransitionsBuilder

[johnpryan]

I recently learned that Material Design apps shouldn't display transition animations on desktop platforms. If this is the case, the default builders in the PageTransitionsTheme class don't seem to be using the correct PageTransitionsBuilder:

  static const Map<TargetPlatform, PageTransitionsBuilder> _defaultBuilders = <TargetPlatform, PageTransitionsBuilder>{
    TargetPlatform.android: FadeUpwardsPageTransitionsBuilder(),
    TargetPlatform.iOS: CupertinoPageTransitionsBuilder(),
    TargetPlatform.linux: FadeUpwardsPageTransitionsBuilder(),
    TargetPlatform.macOS: CupertinoPageTransitionsBuilder(),
    TargetPlatform.windows: FadeUpwardsPageTransitionsBuilder(),
  };

Proposal

Create a new class, NoAnimationPageTransitionsBuilder and use that as the default PageTransitionsBuilder for linux, macOS, and Windows:

class NoAnimationPageTransitionsBuilder extends PageTransitionsBuilder {
  const NoAnimationPageTransitionsBuilder();

  @override
  Widget buildTransitions<T>(
      PageRoute<T> route,
      BuildContext context,
      Animation<double> animation,
      Animation<double> secondaryAnimation,
      Widget child,
      ) {
    return child;
  }
}

[johnpryan]

After more investigation, it looks like this behavior was reverted due to the animation delay: #89997

cc: @darrenaustin WDYT?

[maheshmnj]

Hi @johnpryan, Thanks for filing the issue. This issue is already filed and is being tracked here #99052

Please followup there for further updates, Closing this issue as a duplicate of the linked issue.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.