Skip to content

Conversation

@LinqLover
Copy link
Contributor

image

Also with type guessing:

image

image

Happy Squeaking! 🎈 🎄

@LinqLover
Copy link
Contributor Author

Note to myself: Investigate how thisContext can be moved below self and super.

@LeonMatthes
Copy link
Owner

Note to myself: Investigate how thisContext can be moved below self and super.

Overall this is a nice change. However, sorting thisContext before self and super is a no-go! This needs sorting out before we can merge.

I suspect this may simply be because whatever you used the most recently is sorted to the top.
Because you implemented this completion, it's probably the most recently used in your case, but that shouldn't be the default.
Can you try simply autocompleting self and then checking if self is sorted to the top again?

Otherwise consider overriding ECEntry>>#<= which is responsible for the sorting.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants