stable
Clone or download
Read-only
Part of story #11667 display diff side by side How to test: - Browse a pull request with long lines that need to be wrapped (for example text document) in side-by-side diff - Scrolling stays generally aligned - Given the first line of a file is modified, then a placeholder widget will be added above the "Added line" (instead of below on the previous line) so that scrolling stays aligned Change-Id: I8cdce64daf1ed13518ab0d026263f95e5a6b0a60
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/pullrequest/www/scripts/src/app/app.spec.js | +5 | −3 | Go to diff View file |
M | plugins/pullrequest/www/scripts/src/app/code-collapse/code-collapse-service.spec.js | +0 | −2 | Go to diff View file |
M | plugins/pullrequest/www/scripts/src/app/file-diff/codemirror-helper-service.js | +4 | −3 | Go to diff View file |
M | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-diff-component.js | +28 | −88 | Go to diff View file |
R | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-data-builder.js | Go to diff View file | ||
R | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-data-builder.spec.js | Go to diff View file | ||
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-line-mapper.js | +61 | −0 | Go to diff View file |
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-line-mapper.spec.js | +246 | −0 | Go to diff View file |
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-lines-state.js | +90 | −0 | Go to diff View file |
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-lines-state.spec.js | +103 | −0 | Go to diff View file |
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-widget-builder.js | +130 | −0 | Go to diff View file |
A | plugins/pullrequest/www/scripts/src/app/file-diff/diff-modes/side-by-side-widget-builder.spec.js | +289 | −0 | Go to diff View file |