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


##########
playground/frontend/playground_components/lib/src/widgets/editor_textarea.dart:
##########
@@ -149,15 +149,12 @@ class _EditorTextAreaState extends State<EditorTextArea> {
   }
 
   int _getIndexOfContextLine() {
-    int ctxLineNumber = widget.example!.contextLine;
-    String contextLine = widget.codeController.text.split('\n')[ctxLineNumber];
+    final contextLine = widget.example!.contextLine;

Review Comment:
   Let's do one thing at a time. This is not a regression, there has already 
been `!` here.
   
   Anyway we plan get rid of the numeric context line in the phase 3 and have 
it scrolled to a given hidden comment.



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