Bug 532710: Use better colours for AST View label provider.
updated based on JDT UI coding conventions
Change-Id: Ia77f7cecfd1b7f4db9d0cc1a1b6b93af8702e6e5
diff --git a/org.eclipse.jdt.astview/src/org/eclipse/jdt/astview/views/ASTViewLabelProvider.java b/org.eclipse.jdt.astview/src/org/eclipse/jdt/astview/views/ASTViewLabelProvider.java
index b0b67c1..95607c8 100644
--- a/org.eclipse.jdt.astview/src/org/eclipse/jdt/astview/views/ASTViewLabelProvider.java
+++ b/org.eclipse.jdt.astview/src/org/eclipse/jdt/astview/views/ASTViewLabelProvider.java
@@ -46,7 +46,7 @@
//to dispose:
private final Font fAllocatedBoldItalic;
private final Color fLightRed;
- private Color selectedElemBGColor;
+ private Color fSelectedElemBGColor;
public ASTViewLabelProvider() {
fSelectionStart= -1;
@@ -60,12 +60,12 @@
fDarkGreen= display.getSystemColor(SWT.COLOR_DARK_GREEN);
fDarkRed= display.getSystemColor(SWT.COLOR_DARK_RED);
- selectedElemBGColor= new Color (display, 232, 242, 254);
+ fSelectedElemBGColor= new Color(display, 232, 242, 254);
String currLineColor= EditorsUI.getPreferenceStore().getString(AbstractDecoratedTextEditorPreferenceConstants.EDITOR_CURRENT_LINE_COLOR);
- String [] rgb= currLineColor.split(","); //$NON-NLS-1$
+ String[] rgb= currLineColor.split(","); //$NON-NLS-1$
if (rgb.length == 3) {
try {
- selectedElemBGColor= new Color(display, Integer.parseInt(rgb[0]), Integer.parseInt(rgb[1]), Integer.parseInt(rgb[2]));
+ fSelectedElemBGColor= new Color(display, Integer.parseInt(rgb[0]), Integer.parseInt(rgb[1]), Integer.parseInt(rgb[2]));
} catch (NumberFormatException e) {
// do nothing, colour would remain the backup value
}
@@ -171,7 +171,7 @@
return fLightRed;
}
if (fSelectionStart != -1 && isInside(element)) {
- return selectedElemBGColor;
+ return fSelectedElemBGColor;
}
return null;
}
@@ -239,7 +239,7 @@
@Override
public void dispose() {
super.dispose();
- selectedElemBGColor.dispose();
+ fSelectedElemBGColor.dispose();
fLightRed.dispose();
fAllocatedBoldItalic.dispose();
}
