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

Word wrap doesn't work in Git diffs #20533

Closed
ricksanchez opened this issue Feb 13, 2017 · 4 comments
Closed

Word wrap doesn't work in Git diffs #20533

ricksanchez opened this issue Feb 13, 2017 · 4 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@ricksanchez
Copy link

Word wrap is quite useful features of VS Code in the documents.
However it is broken or not implemented in the git diff view.
This is specially useful here, since the text editor wide is divided by 2.

  • VSCode Version: 1.9.1
  • OS Version: Ubuntu 16.04

Steps to Reproduce:

  1. Edit a document with long lines
  2. go to git ctrl+shift+g
  3. Compare the document
  4. You will see the horizontal scroll bar, and not wrapped text
  5. Press Alt+z (no effect)
@jshall
Copy link

jshall commented Feb 13, 2017

alt+z and editor.wordWrap do not work at all for me in 1.9.0 and 1.9.1 (see #20174), however when in diff mode I've never gotten any of the wrap settings to work (editor.wrappingColumn, editor.wordWrap, or editor.wrappingIndent). I would like to see diff mode support all 3 of these settings.

Currently Using:

  • VS Code Version: 1.9.1
  • OS Version: Windows 7

@Bill-Stewart
Copy link

In my opinion the behavior should be something like the following:

wordWrap wrappingColumn Effect
false any value no wrapping at all
true <= 0 wrap at viewport
true > 0 wrap at specified column

If vscode adopts the behavior in this table, the defaults would be editor.wordWrap: true and editor.wrappingColumn: 300.

I find it non-intuitive and confusing that when wordWrap is false there is sometimes wrapping, and when wordWrap is true there is sometimes not wrapping.

@Bill-Stewart
Copy link

Also, the following extension may be useful:

https://marketplace.visualstudio.com/items?itemName=BillStewart.set-wrapping-column

The purpose of the extension is to let you easily change the wrappingColumn setting without a need to edit the workspace or global settings.json file.

@alexdima
Copy link
Member

Duplicate of #11387

@alexdima alexdima added the *duplicate Issue identified as a duplicate of another issue(s) label Feb 23, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

4 participants