commit 69faf2dbbe26920fba293d4c9adc15cc739c7f0f
author Renan Oliveira <renanoliveira@google.com> Wed Jul 28 22:21:45 2021 +0200
committer Renan Oliveira <renanoliveira@google.com> Wed Jul 28 22:27:40 2021 +0200
tree dcf80350efc6294bcf597981f77f19c2b95c7931
parent 3fadc1db437dae3fed4a829fd7a37b16d97712ef

Don't wrap manually in responsive modes.

We are currently wrapping twice for SHRINK_ONLY mode.

We should ideally leaving wrapping for the browser in these cases
exactly as we do for FULL_RESPONSIVE mode (Fit to screen).

Change-Id: I7c14bdd385134522a1dd37dd5c40e1bfb8a4030b

polygerrit-ui/app/elements/diff/gr-diff-builder/gr-diff-builder.ts

diff --git a/polygerrit-ui/app/elements/diff/gr-diff-builder/gr-diff-builder.ts b/polygerrit-ui/app/elements/diff/gr-diff-builder/gr-diff-builder.ts
index eb50914..864fd79 100644
--- a/polygerrit-ui/app/elements/diff/gr-diff-builder/gr-diff-builder.ts
+++ b/polygerrit-ui/app/elements/diff/gr-diff-builder/gr-diff-builder.ts
@@ -514,9 +514,7 @@
     if (beforeNumber !== 'FILE' && beforeNumber !== 'LOST') {
       const responsiveMode = getResponsiveMode(this._prefs, this._renderPrefs);
       const lineLimit =
-        responsiveMode !== 'FULL_RESPONSIVE'
-          ? this._prefs.line_length
-          : Infinity;
+        responsiveMode === 'NONE' ? this._prefs.line_length : Infinity;
       const contentText = this._formatText(
         line.text,
         this._prefs.tab_size,