alexeyinkin commented on code in PR #23253:
URL: https://github.com/apache/beam/pull/23253#discussion_r971906845
##########
playground/frontend/lib/pages/embedded_playground/components/embedded_actions.dart:
##########
@@ -44,29 +44,29 @@ class EmbeddedActions extends StatelessWidget {
child: SizedBox(
width: kTryPlaygroundButtonWidth,
height: kTryPlaygroundButtonHeight,
- child: Consumer<PlaygroundState>(
- builder: (context, state, child) => ElevatedButton.icon(
+ child: Consumer<PlaygroundController>(
+ builder: (context, controller, child) => ElevatedButton.icon(
icon: SvgPicture.asset(kLinkIconAsset),
label: Text(AppLocalizations.of(context)!.tryInPlayground),
- onPressed: () => _openStandalonePlayground(state),
+ onPressed: () => _openStandalonePlayground(controller),
),
),
),
);
}
- void _openStandalonePlayground(PlaygroundState state) {
+ void _openStandalonePlayground(PlaygroundController controller) {
// The empty list forces the parsing of EmptyExampleLoadingDescriptor
// and prevents the glimpse of the default catalog example.
final window = html.window.open(
- '/?$kExamplesParam=[]&$kSdkParam=${state.sdk?.name}',
Review Comment:
https://medium.com/flutter-senior/never-use-name-as-a-variable-4a18e46fa85c
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
