Sdk: Improve Lambda autocomplete

Created on 5 Mar 2019  路  10Comments  路  Source: dart-lang/sdk

_From @feinstein on March 5, 2019 8:11_

having something like:

myStream.stream.listen()

won't autocomplete into a lambda, which does work for Java and is very handy.

_Copied from original issue: flutter/flutter-intellij#3254_

analyzer-completion analyzer-server area-analyzer

Most helpful comment

Yes, I'd like to see this feature as well.
It was also requested for Flutter.

I have some work in progress, but it is not complete and should be generalized to not only named arguments, and not in a separate completer (probably).
https://dart-review.googlesource.com/c/sdk/+/95481

All 10 comments

_From @feinstein on March 5, 2019 8:25_

/cc @alexander-doroshko

_From @alexander-doroshko on March 5, 2019 8:34_

@feinstein I'm not the right person.
Code completion is powered by the Dart Analysis Server from the Dart SDK. So usually code completion issues are tracked in https://github.com/dart-lang/sdk/issues/, label: area-analyzer. No actions needed, I think Flutter plugin developers will move the issue there if they want to.

BTW I think the issue description could be improved so that developers understand what you do, what you expect and what you get instead.

Thanks for the triage @alexander-doroshko!

@feinstein: I'll move this one over. Cheers for the report!

/cc @scheglov @bwilkerson

Yes, I'd like to see this feature as well.
It was also requested for Flutter.

I have some work in progress, but it is not complete and should be generalized to not only named arguments, and not in a separate completer (probably).
https://dart-review.googlesource.com/c/sdk/+/95481

@scheglov: is this PR abandoned?

Yeah, I started doing additional refactorings to argument completions, and tests. It was taking too much time, and I had more urgent work, so I put this on hold.

@scheglov: do you mind if I pickup where you left off or do you want me to hold off for you to pick this up again?

Sure, you own it now :-)

Implemented w/ e0c178b.

Was this page helpful?
0 / 5 - 0 ratings