opendev
/
gerrit
Code Issues Proposed changes

Change lineWrapping from saying false to using prefs.lineWrapping() 

In I8600778f106 lineWrapping options for edit and diff screens were added.
However, for edit screen, the default was missed to be set to the new option.
It was still set to false, so that the value of new option was ignored.

Reported-By: David Ostrovsky <david.ostrovsky@gmail.com>
Change-Id: Ibd1315a3e95ae01bb830a140c3113df0262ef072
This commit is contained in:
Paladox none 2017-07-01 13:38:05 +00:00
parent 262f893349
commit 6e3ed27ec2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/editor/EditScreen.java
View File

@ -565,7 +565,7 @@ public class EditScreen extends Screen {
.set("indentUnit", prefs.indentUnit())
.set("keyMap", prefs.keyMapType().name().toLowerCase())
.set("lineNumbers", prefs.hideLineNumbers())
.set("lineWrapping", false)
.set("lineWrapping", prefs.lineWrapping())
.set("matchBrackets", prefs.matchBrackets())
.set("mode", mode != null ? mode.mime() : null)
.set("origLeft", editContent)