alexeyinkin commented on code in PR #22735:
URL: https://github.com/apache/beam/pull/22735#discussion_r949875663


##########
playground/frontend/lib/pages/embedded_playground/components/embedded_editor.dart:
##########
@@ -29,11 +31,17 @@ class EmbeddedEditor extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     final state = Provider.of<PlaygroundState>(context);
+    final controller = state.snippetEditingController;
+
+    if (controller == null) {
+      return const LoadingIndicator(size: kLgLoadingIndicatorSize);

Review Comment:
   We used to show a blank editor before the example is loaded, and it was 
confusing. Now we only get the controller with the example already in it, 
unless deliberately showing an empty editor.



-- 
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]

Reply via email to