Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the eldoc for interop forms #1725

Closed
bbatsov opened this issue Apr 30, 2016 · 2 comments
Closed

Improve the eldoc for interop forms #1725

bbatsov opened this issue Apr 30, 2016 · 2 comments

Comments

@bbatsov
Copy link
Member

bbatsov commented Apr 30, 2016

Currently the eldoc for interop forms is not very clear. We simply display the name of the method and all possible arglists from all matching classes. I'm reasonably sure not all users are aware of this.

We can improve this a bit in the following manner:

  • display the class name when the method/field in question is present in just one class
  • display the number of matching classes (or their names) before the method name in cases of ambiguities

Some the end results should be something like "SomeClass/someMethod: ..." and "(n classes)/someMethod: ...".

@ckoparkar
Copy link
Contributor

I'd like to give this a try. I'll submit a PR soon.

@bbatsov
Copy link
Member Author

bbatsov commented May 3, 2016

Great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants