opendev/gerrit
Code Issues Proposed changes

Fix bug where inline diffs are incorrect when jumping changes

Browse Source 
Collapse them upon reload of the file list.

Change-Id: I2b0e115176db83169324f2d9289aae2665886f8c
This commit is contained in:
Andrew Bonventre
2016-05-17 16:46:23 -07:00
parent 251443314d
commit fafca2f12b
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.html
View File

@@ -168,7 +168,7 @@ limitations under the License.
<span class="removed">-[[file.lines_deleted]]</span> <span class="removed">-[[file.lines_deleted]]</span>
</div> </div>
</div> </div>
<gr-diff id="diff" hidden <gr-diff hidden
change-num="[[changeNum]]" change-num="[[changeNum]]"
patch-range="[[patchRange]]" patch-range="[[patchRange]]"
path="[[file.__path]]" path="[[file.__path]]"

2
polygerrit-ui/app/elements/change/gr-file-list/gr-file-list.js
View File

@@ -60,6 +60,8 @@
return Promise.resolve(); return Promise.resolve();
} }
this._collapseAllDiffs();
var promises = []; var promises = [];
var _this = this; var _this = this;