opendev/gerrit
Code Issues Proposed changes

Permit use of syntax highlighting in unified view

Browse Source 
Users might actually want to enable the syntax highlighting in the
unified view, even though its not the way we want to present lines
to them.  Leave it off by default, but make the checkbox available
so it can be turned on if desired.

Change-Id: I9558a872190408b5ef47412933199e5d608fe7cb
Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2010-03-04 17:13:29 -08:00
parent ce4a59b0a8
commit a6658ca2d4
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java
View File

@@ -45,7 +45,6 @@ import com.google.gwt.event.logical.shared.ValueChangeEvent;
import com.google.gwt.event.logical.shared.ValueChangeHandler;
import com.google.gwt.event.shared.HandlerRegistration;
import com.google.gwt.user.client.Command;
import com.google.gwt.user.client.DOM;
import com.google.gwt.user.client.DeferredCommand;
import com.google.gwt.user.client.rpc.AsyncCallback;
import com.google.gwt.user.client.ui.DisclosurePanel;
@@ -86,9 +85,6 @@ public abstract class PatchScreen extends Screen implements
public Unified(final Patch.Key id, final int patchIndex,
final PatchTable patchTable) {
super(id, patchIndex, patchTable);
DOM.setStyleAttribute(settingsPanel.getSyntaxHighlightingCheckBox()
.getElement(), "visibility", "hidden");
final PatchScriptSettings s = settingsPanel.getValue();
s.getPrettySettings().setSyntaxHighlighting(false);
settingsPanel.setValue(s);