[jbosstools-commits] JBoss Tools SVN: r42833 - in trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor: util and 1 other directory.

[jbosstools-commits at lists.jboss.org]

Author: dmaliarevich
Date: 2012-08-02 07:25:51 -0400 (Thu, 02 Aug 2012)
New Revision: 42833

Modified:
   trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/VpeColorSelector.java
   trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java
Log:
https://issues.jboss.org/browse/JBIDE-12395 - default value is loaded automatically from the PreferencesStore.

Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/VpeColorSelector.java
===================================================================
--- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/VpeColorSelector.java	2012-08-02 10:57:34 UTC (rev 42832)
+++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/VpeColorSelector.java	2012-08-02 11:25:51 UTC (rev 42833)
@@ -25,7 +25,6 @@
 		colorText = new Text(parent, SWT.NONE);
 		colorText.setEditable(false);
 		colorText.setTextLimit(9);
-		colorText.setText(VpeStyleUtil.rgbToString(getColorValue()));
 	}
 	
 	@Override

Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java
===================================================================
--- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java	2012-08-02 10:57:34 UTC (rev 42832)
+++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java	2012-08-02 11:25:51 UTC (rev 42833)
@@ -994,7 +994,7 @@
 					+ Integer.toHexString(rgb.blue);
 			colorStr = colorStr.toUpperCase();
 		} else {
-			VpePlugin.getDefault().logError("Cannot convert RGB color to string, because it is null"); //$NON-NLS-1$
+			VpePlugin.getDefault().logWarning("VpeStyleUtil.rgbToString(RGB rgb) -> Cannot convert RGB color to string, because it is null"); //$NON-NLS-1$
 		}
 		return colorStr;
 	}