search

abrookins / WrapToColumn

An IntelliJ plugin that wraps text
Apache License 2.0
62 stars 12 forks source link

Remove separate shortcuts #32

Closed viliam-durina closed 4 years ago

viliam-durina commented 4 years ago

The plugin used to have 2 shortcuts: one to auto-select current paragraph and reflow it and the other to reflow selected text. But this can be decided based on the presence of a selection: if a text is selected, format the selection, otherwise format the current paragraph.

If there wasn't a selection and "format selection" was used, the current line was selected and formatted, which isn't very intuitive and I never used it.

abrookins commented 4 years ago

Agreed - better to simplify to one command. Looks like a nice improvement!

viliam-durina commented 4 years ago

Glad you agreed, I have few more changes ready, will submit few more PRs. Then you can maybe do a release.