Too Agressive Add {...} to arrow function about js-assistant HOT 5 CLOSED

That'd be great!

from js-assistant.

Thanks for the bug report!

I'm trying to understand what the bug is here. Add to arrow function should be available, as it seems to be a legit code action, and it does not seem to be suggested.

Are you looking for a better activation range in the text? Or is it about the action showing up as a quick-fix?

from js-assistant.

The cursor is in the nested expression, thus I would not expect this code action.

In other cases, this code action hides a quick fix.

Maybe it makes sense to only offer this code action on the arrow.

from js-assistant.

That makes total sense. I'm thinking about having 2 levels of activation:

• quick fix on the function head and the arrow and the space after, i.e., on () => here.
• code action (refactoring menu, convert menu) on the function it is now

Would this option work for you?

from js-assistant.

I've tweaked the activation ranges for 'add/remove {}' in v1.120.2. Thanks for the feedback!

from js-assistant.