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

Wrap Lines ST4 regression #3177

Closed
gerardroche opened this issue Jan 16, 2020 · 1 comment
Closed

Wrap Lines ST4 regression #3177

gerardroche opened this issue Jan 16, 2020 · 1 comment

Comments

@gerardroche
Copy link

gerardroche commented Jan 16, 2020

The Wrap Lines command, since 4061, wraps entire blocks or "paragraphs". Previously if text was selected the wrap lines command only wrap the selected text, otherwise it would wrap the paragraph.

The API command is wrap_lines.

In the UI it's Menu > Edit > Wrap > Wrap Paragraph at Ruler.

Steps to reproduce

Set a wrap small wrap width to make it easier to test:

>>> sublime.active_window().active_view().settings().set('wrap_width', 5)`

Make a selection (where |...| is the selection):

  xx xx xx
  a1 |a2 a3|
  x1 x2 x3
  b1 b2 b3

Run the wrap lines command:

>>> sublime.active_window().active_view().run_command('wrap_lines')

Expected behavior

The wrap_lines commands should only wrap the selected text, not the entire paragraph.

  xx xx xx
  a1
  |a2
  a3|
  x1 x2 x3
  b1 b2 b3

Actual behavior

The wrap_lines wraps the entire block, instead of just the selection.

  xx
  xx
  xx
  a1
  |a2
  a3|
  x1
  x2
  x3
  b1
  b2
  b3

Environment

  • Sublime Text Build: >=4061
  • Operating system and version: Ubuntu 18.04...20.04
@gerardroche gerardroche changed the title wrap_lines command doesn't work for selections in 4061 Wrap Lines doesn't work for selections in 4061 (regression) May 1, 2020
@gerardroche gerardroche changed the title Wrap Lines doesn't work for selections in 4061 (regression) Wrap Lines regression >=4061 Aug 22, 2020
gerardroche added a commit to NeoVintageous/NeoVintageous that referenced this issue Sep 23, 2020
@gerardroche gerardroche changed the title Wrap Lines regression >=4061 Wrap Lines ST4 regression Dec 23, 2021
@FichteFoll
Copy link
Collaborator

I can only assume that this was a more or less intentional change.

@FichteFoll FichteFoll closed this as not planned Won't fix, can't repro, duplicate, stale Oct 9, 2023
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