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

To Arrow Anchor Point #51

Closed
hediet opened this issue Sep 1, 2022 · 6 comments
Closed

To Arrow Anchor Point #51

hediet opened this issue Sep 1, 2022 · 6 comments

Comments

@hediet
Copy link

hediet commented Sep 1, 2022

image

Wasn't the "Remove {...} from arrow function" action suggested here once?

@lgrammel
Copy link
Contributor

lgrammel commented Sep 1, 2022

Thanks, I'll take a look!

@lgrammel
Copy link
Contributor

lgrammel commented Sep 2, 2022

It is available as a regular code action (convert), but not as a quickfix at that position. Were you expecting a quickfix (I could increase the quickfix range)?

@hediet
Copy link
Author

hediet commented Sep 2, 2022

Yep, was expecting a quick fix there

@lgrammel
Copy link
Contributor

lgrammel commented Sep 2, 2022

Cool, will change it in the next update. Btw, if you want to add/remove braces even faster, you can try the toggle braces shortcut (Ctrl + Alt + B on Windows).

@hediet
Copy link
Author

hediet commented Sep 2, 2022

I already use quite a few shortcuts and would be happy if I could remember the one for organize imports...

@lgrammel
Copy link
Contributor

lgrammel commented Sep 2, 2022

I have trouble with that one as well for some reason.

"Remove braces from arrow function" is available as quick fix on the "return" keyword starting with v1.147.1 - thanks for the suggestion!

@lgrammel lgrammel closed this as completed Sep 2, 2022
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

No branches or pull requests

2 participants