Move to Java5

Fix malformed JavaDocs.

116 files changed, 528 insertions, 137 deletions

diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.classpath b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.classpath
index 751c8f2e5..304e86186 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.classpath
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.classpath
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
 	<classpathentry kind="src" path="src"/>
-	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
+	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/J2SE-1.5"/>
 	<classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/>
 	<classpathentry kind="output" path="bin"/>
 </classpath>
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.settings/org.eclipse.jdt.core.prefs b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.settings/org.eclipse.jdt.core.prefs
index a92ec7bad..ff85cd061 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.settings/org.eclipse.jdt.core.prefs
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/.settings/org.eclipse.jdt.core.prefs
@@ -1,9 +1,9 @@
-#Sat Apr 07 15:19:56 PDT 2007
+#Tue Apr 10 10:08:32 PDT 2007
 eclipse.preferences.version=1
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=disabled
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.4
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.5
 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
-org.eclipse.jdt.core.compiler.compliance=1.4
+org.eclipse.jdt.core.compiler.compliance=1.5
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -16,7 +16,7 @@ org.eclipse.jdt.core.compiler.problem.deprecationInDeprecatedCode=disabled
 org.eclipse.jdt.core.compiler.problem.deprecationWhenOverridingDeprecatedMethod=disabled
 org.eclipse.jdt.core.compiler.problem.discouragedReference=ignore
 org.eclipse.jdt.core.compiler.problem.emptyStatement=ignore
-org.eclipse.jdt.core.compiler.problem.enumIdentifier=warning
+org.eclipse.jdt.core.compiler.problem.enumIdentifier=error
 org.eclipse.jdt.core.compiler.problem.fallthroughCase=ignore
 org.eclipse.jdt.core.compiler.problem.fatalOptionalError=enabled
 org.eclipse.jdt.core.compiler.problem.fieldHiding=warning
@@ -27,7 +27,7 @@ org.eclipse.jdt.core.compiler.problem.hiddenCatchBlock=warning
 org.eclipse.jdt.core.compiler.problem.incompatibleNonInheritedInterfaceMethod=warning
 org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=warning
 org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=ignore
-org.eclipse.jdt.core.compiler.problem.invalidJavadoc=warning
+org.eclipse.jdt.core.compiler.problem.invalidJavadoc=error
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=disabled
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=disabled
@@ -51,14 +51,14 @@ org.eclipse.jdt.core.compiler.problem.overridingPackageDefaultMethod=warning
 org.eclipse.jdt.core.compiler.problem.parameterAssignment=ignore
 org.eclipse.jdt.core.compiler.problem.possibleAccidentalBooleanAssignment=warning
 org.eclipse.jdt.core.compiler.problem.potentialNullReference=ignore
-org.eclipse.jdt.core.compiler.problem.rawTypeReference=warning
+org.eclipse.jdt.core.compiler.problem.rawTypeReference=ignore
 org.eclipse.jdt.core.compiler.problem.redundantNullCheck=ignore
 org.eclipse.jdt.core.compiler.problem.specialParameterHidingField=disabled
 org.eclipse.jdt.core.compiler.problem.staticAccessReceiver=warning
 org.eclipse.jdt.core.compiler.problem.suppressWarnings=enabled
 org.eclipse.jdt.core.compiler.problem.syntheticAccessEmulation=ignore
 org.eclipse.jdt.core.compiler.problem.typeParameterHiding=ignore
-org.eclipse.jdt.core.compiler.problem.uncheckedTypeOperation=warning
+org.eclipse.jdt.core.compiler.problem.uncheckedTypeOperation=ignore
 org.eclipse.jdt.core.compiler.problem.undocumentedEmptyBlock=error
 org.eclipse.jdt.core.compiler.problem.unhandledWarningToken=warning
 org.eclipse.jdt.core.compiler.problem.unnecessaryElse=warning
@@ -74,4 +74,4 @@ org.eclipse.jdt.core.compiler.problem.unusedParameterWhenImplementingAbstract=di
 org.eclipse.jdt.core.compiler.problem.unusedParameterWhenOverridingConcrete=disabled
 org.eclipse.jdt.core.compiler.problem.unusedPrivateMember=error
 org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=warning
-org.eclipse.jdt.core.compiler.source=1.4
+org.eclipse.jdt.core.compiler.source=1.5
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/META-INF/MANIFEST.MF b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/META-INF/MANIFEST.MF
index a853645dd..37c125ef8 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/META-INF/MANIFEST.MF
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/META-INF/MANIFEST.MF
@@ -44,3 +44,4 @@ Export-Package: org.eclipse.jst.pagedesigner.jsf.core.dom,
  org.eclipse.jst.pagedesigner.jsf.ui.elementedit.util,
  org.eclipse.jst.pagedesigner.jsf.ui.sections,
  org.eclipse.jst.pagedesigner.jsf.ui.util
+Bundle-RequiredExecutionEnvironment: J2SE-1.5
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/build.properties b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/build.properties
index 06c4a0ce7..cdcce24a6 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/build.properties
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/build.properties
@@ -8,3 +8,5 @@ bin.includes = plugin.xml,\
                plugin.properties,\
                metadata/
 jars.compile.order = .
+javacSource=1.5
+javacTarget=1.5 
\ No newline at end of file
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/core/dom/JSFDOMUtil.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/core/dom/JSFDOMUtil.java
index 1f8afd120..16992f70f 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/core/dom/JSFDOMUtil.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/core/dom/JSFDOMUtil.java
@@ -29,9 +29,9 @@ public class JSFDOMUtil
 {
 
     /**
-     * @param panelGridEle
-     * @param string
-     * @return
+     * @param parentEle 
+     * @param name 
+     * @return the facet element or null if not found
      */
     public static Element findFacet(Element parentEle, String name)
     {
@@ -51,6 +51,10 @@ public class JSFDOMUtil
         return null;
     }
 
+    /**
+     * @param node
+     * @return true if node is a JSF column
+     */
     public static boolean isHColumn(Node node)
     {
         if (node instanceof Element)
@@ -67,7 +71,7 @@ public class JSFDOMUtil
 
     /**
      * @param panelGridEle
-     * @return
+     * @return the list of UI component children
      */
     public static List getUIComponentChildren(Element panelGridEle)
     {
@@ -90,6 +94,10 @@ public class JSFDOMUtil
         return list;
     }
 
+    /**
+     * @param ele
+     * @return true if ele is a facet
+     */
     public static boolean isFacet(Element ele)
     {
         // XXX: here we are not checking namespace! for error tolerant.
@@ -97,8 +105,8 @@ public class JSFDOMUtil
     }
 
     /**
-     * @param child
-     * @return
+     * @param ele 
+     * @return true if the local name of ele is "param"
      */
     public static boolean isUIParameter(Element ele)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/JSFUIPlugin.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/JSFUIPlugin.java
index de9e382ac..ec0e9dd29 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/JSFUIPlugin.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/JSFUIPlugin.java
@@ -74,7 +74,7 @@ public class JSFUIPlugin extends AbstractUIPlugin
 
     /**
      * get the alerts objects associated with this plugin for alerting the user.
-     * @return
+     * @return the alerts
      */
     public static Alerts getAlerts()
     {
@@ -91,6 +91,7 @@ public class JSFUIPlugin extends AbstractUIPlugin
 
     /**
      * Returns the shared instance.
+     * @return the default plugin
      */
     public static JSFUIPlugin getDefault()
     {
@@ -100,6 +101,8 @@ public class JSFUIPlugin extends AbstractUIPlugin
     /**
      * Returns the string from the plugin's resource bundle,
      * or 'key' if not found.
+     * @param key 
+     * @return the resource string for key or 'key' if not found
      */
     public static String getResourceString(String key)
     {
@@ -116,6 +119,7 @@ public class JSFUIPlugin extends AbstractUIPlugin
 
     /**
      * Returns the plugin's resource bundle,
+     * @return the resource bundle
      */
     public ResourceBundle getResourceBundle()
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ActionsResources.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ActionsResources.java
index bfa8dd826..77ee60bf8 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ActionsResources.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ActionsResources.java
@@ -29,9 +29,12 @@ public class ActionsResources
         // no external instantiation
     }
 
+    /**
+     * @param key
+     * @return the res bundle string for key or !key! if not found
+     */
     public static String getString(String key)
     {
-        // TODO Auto-generated method stub
         try
         {
             return RESOURCE_BUNDLE.getString(key);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddActionListenerAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddActionListenerAction.java
index a3b7731dc..131c134ab 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddActionListenerAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddActionListenerAction.java
@@ -24,6 +24,7 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.DialogUtil;
 public class AddActionListenerAction extends JSFAddChildAction
 {
     /**
+     * @param parentEle 
      * 
      */
     public AddActionListenerAction(IDOMElement parentEle)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddAttributeAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddAttributeAction.java
index 6eceabb6e..2942f285a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddAttributeAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddAttributeAction.java
@@ -24,7 +24,6 @@ public class AddAttributeAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddAttributeAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertDateTimeAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertDateTimeAction.java
index dacf02ea9..6172669fa 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertDateTimeAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertDateTimeAction.java
@@ -24,7 +24,6 @@ public class AddConvertDateTimeAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddConvertDateTimeAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertNumberAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertNumberAction.java
index 8670f060a..af33b5245 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertNumberAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConvertNumberAction.java
@@ -24,7 +24,6 @@ public class AddConvertNumberAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddConvertNumberAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConverterAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConverterAction.java
index 58ece5497..67b2e3085 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConverterAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddConverterAction.java
@@ -28,7 +28,7 @@ public class AddConverterAction extends JSFAddChildAction
     String	_converterId;
 
     /**
-     * @param text
+     * @param converterId 
      * @param parentNode
      */
     public AddConverterAction(String converterId, IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddParamAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddParamAction.java
index ff8876d2d..8cb1a7419 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddParamAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddParamAction.java
@@ -25,7 +25,6 @@ public class AddParamAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddParamAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemAction.java
index eb7f3dba4..c725801f2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemAction.java
@@ -25,7 +25,6 @@ public class AddSelectItemAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddSelectItemAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemsAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemsAction.java
index 42e8527d6..c4ae73c07 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemsAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddSelectItemsAction.java
@@ -23,6 +23,9 @@ import org.eclipse.wst.xml.core.internal.provisional.document.IDOMElement;
 public class AddSelectItemsAction extends JSFAddChildAction
 {
 
+    /**
+     * @param parentNode
+     */
     public AddSelectItemsAction(IDOMElement parentNode)
     {
         super(ActionsResources.getString("AddSelectItemsAction.ActionLabel.SelectItems"), parentNode); //$NON-NLS-1$
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateDoubleRangeAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateDoubleRangeAction.java
index 77d4425d1..c6c785562 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateDoubleRangeAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateDoubleRangeAction.java
@@ -25,7 +25,6 @@ public class AddValidateDoubleRangeAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddValidateDoubleRangeAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLengthAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLengthAction.java
index 186ea86fb..b89cf1d12 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLengthAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLengthAction.java
@@ -25,7 +25,6 @@ public class AddValidateLengthAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddValidateLengthAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLongRangeAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLongRangeAction.java
index 2f0b716f9..a12ce56eb 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLongRangeAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValidateLongRangeAction.java
@@ -25,7 +25,6 @@ public class AddValidateLongRangeAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddValidateLongRangeAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValueChangeListenerAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValueChangeListenerAction.java
index f13ccd478..cb76f4489 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValueChangeListenerAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/AddValueChangeListenerAction.java
@@ -25,7 +25,6 @@ public class AddValueChangeListenerAction extends JSFAddChildAction
 {
 
     /**
-     * @param text
      * @param parentNode
      */
     public AddValueChangeListenerAction(IDOMElement parentNode)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHColumnHeaderFooterAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHColumnHeaderFooterAction.java
index 58955c176..8d6fa01d4 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHColumnHeaderFooterAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHColumnHeaderFooterAction.java
@@ -27,6 +27,8 @@ public class DeleteHColumnHeaderFooterAction extends Action
 
     /**
      * @param text
+     * @param editPart 
+     * @param isHeader 
      */
     public DeleteHColumnHeaderFooterAction(String text, EditPart editPart, boolean isHeader)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHeaderFooterAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHeaderFooterAction.java
index d3f03f42e..7df470c16 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHeaderFooterAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/DeleteHeaderFooterAction.java
@@ -27,6 +27,8 @@ public class DeleteHeaderFooterAction extends Action
 
     /**
      * @param text
+     * @param editPart 
+     * @param isHeader 
      */
     public DeleteHeaderFooterAction(String text, EditPart editPart, boolean isHeader)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ExpressionAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ExpressionAction.java
index fe66f8867..78f1782fc 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ExpressionAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/ExpressionAction.java
@@ -28,7 +28,13 @@ import org.eclipse.ui.PlatformUI;
  */
 public class ExpressionAction extends Action
 {
+    /**
+     * enumerates a method-binding expression
+     */
     public static final int METHOD   = 0;
+    /**
+     * enumerates a value-binding expression
+     */
     public static final int VARIABLE = 1;
 
     private String          _action;
@@ -36,6 +42,9 @@ public class ExpressionAction extends Action
     private IFile           _file;
     private int             _type;
 
+    /**
+     * @param type
+     */
     public ExpressionAction(int type)
     {
         _type = type;
@@ -108,11 +117,17 @@ public class ExpressionAction extends Action
         return window.getActivePage();
     }
 
+    /**
+     * @param fileFor
+     */
     public void setFile(IFile fileFor)
     {
         _file = fileFor;
     }
 
+    /**
+     * @return the file
+     */
     public IFile getFile()
     {
         return _file;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/IJSFCoreSupport.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/IJSFCoreSupport.java
index 78752f2ae..70285ee83 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/IJSFCoreSupport.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/IJSFCoreSupport.java
@@ -26,31 +26,31 @@ public interface IJSFCoreSupport
 {
     /**
      * actionListener can only be added to those ActionSource component.
-     * @return
+     * @return if is an action source
      */
     public boolean isActionSource();
 
     /**
      * 
-     * @return
+     * @return true if is a UI component
      */
     public boolean isUIComponent();
 
     /**
      * converter (convertXXX) can only be added to those ValueHolder
-     * @return
+     * @return true if is a value holder
      */
     public boolean isValueHolder();
 
     /**
      * validator can only be added to EditableValueHolder
-     * @return
+     * @return true if is an editable value holder
      */
     public boolean isEditableValueHolder();
 
     /**
      * whether support selectItem and selectItems
-     * @return
+     * @return true if supports the UISelectItem(s) contract
      */
     public boolean supportSelectItems();
 }
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHColumnHeaderFooterAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHColumnHeaderFooterAction.java
index d6e3f512b..e17721259 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHColumnHeaderFooterAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHColumnHeaderFooterAction.java
@@ -27,6 +27,8 @@ public class InsertHColumnHeaderFooterAction extends Action
 
     /**
      * @param text
+     * @param editPart 
+     * @param isHeader 
      */
     public InsertHColumnHeaderFooterAction(String text, EditPart editPart, boolean isHeader)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHeaderFooterAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHeaderFooterAction.java
index 3f41adb45..f2ac6b823 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHeaderFooterAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/InsertHeaderFooterAction.java
@@ -27,6 +27,8 @@ public class InsertHeaderFooterAction extends Action
 
     /**
      * @param text
+     * @param editPart 
+     * @param isHeader 
      */
     public InsertHeaderFooterAction(String text, EditPart editPart, boolean isHeader)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
index 9a866b20f..efef755d9 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
@@ -41,10 +41,11 @@ public class JSFAddActionGroup {
         // TODO: why?
 	};
 
-	/*
-	 * (non-Javadoc)
-	 * 
-	 * @see org.eclipse.ui.actions.ActionGroup#fillContextMenu(org.eclipse.jface.action.IMenuManager)
+
+	/**
+	 * @param menu
+	 * @param element
+	 * @param support
 	 */
 	public void fillContextMenu(IMenuManager menu, final IDOMElement element,
 			final IJSFCoreSupport support) {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddChildAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddChildAction.java
index 32167adb1..058f67e5c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddChildAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddChildAction.java
@@ -29,7 +29,8 @@ public class JSFAddChildAction extends Action
     private IDOMElement _parentEle;
 
     /**
-     * 
+     * @param text 
+     * @param parentNode 
      */
     public JSFAddChildAction(String text, IDOMElement parentNode)
     {
@@ -37,11 +38,17 @@ public class JSFAddChildAction extends Action
         this._parentEle = parentNode;
     }
 
+    /**
+     * @return the parent element
+     */
     public IDOMElement getParentElement()
     {
         return _parentEle;
     }
 
+    /**
+     * @return the active shell
+     */
     public Shell getShell()
     {
         IWorkbench bench = PlatformUI.getWorkbench();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFCommandLinkCreator.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFCommandLinkCreator.java
index 9012bc4a7..41d0a3ae2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFCommandLinkCreator.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFCommandLinkCreator.java
@@ -67,10 +67,12 @@ public class JSFCommandLinkCreator extends AbstractLinkCreator
 
     }
 
-    /* (non-Javadoc)
-     * @see org.eclipse.jst.pagedesigner.actions.link.ILinkCreator#canExcute(org.eclipse.jst.pagedesigner.viewer.DesignRange)
+
+    /**
+     * @param range
+     * @return true if can execute
      */
-    public boolean canExcute(DesignRange range)
+    public boolean canExecute(DesignRange range)
     {
         DesignPosition startPosition = range.getStartPosition();
         EditPart part = startPosition.getContainerPart();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFNavigationGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFNavigationGroup.java
index 2e277b232..8f2e179e5 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFNavigationGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFNavigationGroup.java
@@ -35,8 +35,11 @@ public class JSFNavigationGroup
         // TODO: why?
     };
 
-    /* (non-Javadoc)
-     * @see org.eclipse.ui.actions.ActionGroup#fillContextMenu(org.eclipse.jface.action.IMenuManager)
+
+    /**
+     * @param menu
+     * @param element
+     * @param support
      */
     public void fillContextMenu(IMenuManager menu, final IDOMElement element, final IJSFCoreSupport support)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFOutputLinkCreator.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFOutputLinkCreator.java
index ff8380106..036f44525 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFOutputLinkCreator.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFOutputLinkCreator.java
@@ -69,9 +69,9 @@ public class JSFOutputLinkCreator extends AbstractLinkCreator
     }
 
     /* (non-Javadoc)
-     * @see org.eclipse.jst.pagedesigner.actions.link.ILinkCreator#canExcute(org.eclipse.jst.pagedesigner.viewer.DesignRange)
+     * @see org.eclipse.jst.pagedesigner.actions.link.AbstractLinkCreator#canExecute(org.eclipse.jst.pagedesigner.viewer.DesignRange)
      */
-    public boolean canExcute(DesignRange range)
+    public boolean canExecute(DesignRange range)
     {
         DesignPosition startPosition = range.getStartPosition();
         EditPart part = startPosition.getContainerPart();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/PageFlowContributor.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/PageFlowContributor.java
index c175805f6..f43db4ff5 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/PageFlowContributor.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/PageFlowContributor.java
@@ -74,14 +74,23 @@ public class PageFlowContributor implements IContextMenuItemContributor
 
     class PageFlowAction extends Action
     {
+        /**
+         * this action's id
+         */
         public final static String ID = "org.eclipse.jst.pagedesigner.jsf.ui.actions.PageFlowAction"; //$NON-NLS-1$
         private IStructuredModel   model;
 
+        /**
+         * construct the action
+         */
         public PageFlowAction()
         {
             setText(ActionsResources.getString("PageFlowContributor.Menu.PageFlow"));//$NON-NLS-1$
         }
 
+        /**
+         * @param model
+         */
         public void setModel(IStructuredModel model)
         {
             this.model = model;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/CommonDialogFieldWizard.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/CommonDialogFieldWizard.java
index 8e33a9cbb..03dcb90a5 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/CommonDialogFieldWizard.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/CommonDialogFieldWizard.java
@@ -21,6 +21,11 @@ import org.eclipse.ui.internal.dialogs.NewWizard;
 public class CommonDialogFieldWizard extends NewWizard
 {
     DialogFieldGroupPage mainPage;
+    /**
+     * Constructg a new wizard using page as the wizard mage
+     * 
+     * @param page
+     */
     public CommonDialogFieldWizard(DialogFieldGroupPage page)
     {
         this.mainPage = page;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/DialogFieldFactory.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/DialogFieldFactory.java
index a7c7b8845..a9392b8ef 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/DialogFieldFactory.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/attributegroup/DialogFieldFactory.java
@@ -29,6 +29,10 @@ import org.eclipse.jst.pagedesigner.meta.internal.CMRegistry;
  */
 public class DialogFieldFactory
 {
+    /**
+     * @param data
+     * @return the dialog field for data
+     */
     public static DialogField getDialogField(AttributeData data)
     {
         IAttributeDescriptor descriptor = getAttributeDescriptor(data.uri, data.elementName, data.attributeName);
@@ -58,6 +62,10 @@ public class DialogFieldFactory
         return elementDescriptor.getAttributeDescriptor(attributeName);
     }
 
+    /**
+     * @param field
+     * @param value
+     */
     public static void setDialogFiledValue(DialogField field, Object value)
     {
         if (field instanceof StringDialogField)
@@ -90,9 +98,8 @@ public class DialogFieldFactory
     }
 
     /**
-     * @param element
-     * @param string
-     * @return
+     * @param data
+     * @return the dialog field label for data
      */
     public static String getDialogFieldLabel(AttributeData data)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/celleditors/ConverterValidatorIdDialogField.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/celleditors/ConverterValidatorIdDialogField.java
index 0e83c1461..07c7ce30d 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/celleditors/ConverterValidatorIdDialogField.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/celleditors/ConverterValidatorIdDialogField.java
@@ -28,7 +28,7 @@ public class ConverterValidatorIdDialogField extends ComboDialogField implements
     private String _elementId = "";
 
     /**
-     * @param flags
+     * The default constructor
      */
     public ConverterValidatorIdDialogField()
     {
@@ -61,11 +61,19 @@ public class ConverterValidatorIdDialogField extends ComboDialogField implements
         }
     }
 
+    /**
+     * @return the element id
+     */
     public String getElementId()
     {
         return _elementId;
     }
 
+    /**
+     * Set the element id
+     * 
+     * @param elementId
+     */
     public void setElementId(String elementId)
     {
         this._elementId = elementId;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/AddJSFCoreChildCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/AddJSFCoreChildCommand.java
index 34f076842..70eec6413 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/AddJSFCoreChildCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/AddJSFCoreChildCommand.java
@@ -38,7 +38,9 @@ public class AddJSFCoreChildCommand extends DesignerCommand
     private IDOMElement _parent;
 
     /**
-     * 
+     * @param parent 
+     * @param coreChildLocalName 
+     * @param attributes 
      */
     public AddJSFCoreChildCommand(IDOMElement parent, String coreChildLocalName, Map attributes)
     {
@@ -56,10 +58,10 @@ public class AddJSFCoreChildCommand extends DesignerCommand
     }
 
     /**
-     * @param parentElement
-     * @param string
-     * @param string2
-     * @param newClassName
+     * @param parent 
+     * @param coreChildLocalName 
+     * @param aname 
+     * @param avalue 
      */
     public AddJSFCoreChildCommand(IDOMElement parent, String coreChildLocalName, String aname, String avalue)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/CommandResources.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/CommandResources.java
index 147bc3912..11e8976ad 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/CommandResources.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/CommandResources.java
@@ -29,6 +29,10 @@ public final class CommandResources
         // no external instantiation
     }
 
+    /**
+     * @param key
+     * @return the resource for key or !key! if not found
+     */
     public static String getString(String key)
     {
         try
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnCommand.java
index 799c99e6d..12dce24e1 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnCommand.java
@@ -33,8 +33,9 @@ public class DataTableDeleteColumnCommand extends DesignerCommand
     private int     _index;
 
     /**
-     * @param label
      * @param viewer
+     * @param dataTable 
+     * @param columnIndex 
      */
     public DataTableDeleteColumnCommand(IHTMLGraphicalViewer viewer, Element dataTable, int columnIndex)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnHeaderCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnHeaderCommand.java
index 9e97d742a..c16d4eb5a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnHeaderCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteColumnHeaderCommand.java
@@ -30,8 +30,9 @@ public class DataTableDeleteColumnHeaderCommand extends DesignerCommand
     private Element _dataTable;
 
     /**
-     * @param label
      * @param viewer
+     * @param dataTable 
+     * @param header 
      */
     public DataTableDeleteColumnHeaderCommand(IHTMLGraphicalViewer viewer, Element dataTable, boolean header)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteHeaderCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteHeaderCommand.java
index 727359f00..44dfff205 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteHeaderCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableDeleteHeaderCommand.java
@@ -30,6 +30,11 @@ public class DataTableDeleteHeaderCommand extends DesignerCommand
     boolean         _isHeader;
     private Element _dataTable;
 
+    /**
+     * @param viewer
+     * @param dataTable
+     * @param isHeader
+     */
     public DataTableDeleteHeaderCommand(IHTMLGraphicalViewer viewer, Element dataTable, boolean isHeader)
     {
         super(isHeader ? CommandResources.getString("DataTableDeleteHeaderCommand.Label.DeleteHeader") : //$NON-NLS-1$
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertColumnHeaderCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertColumnHeaderCommand.java
index d13c5ce95..f69707ff4 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertColumnHeaderCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertColumnHeaderCommand.java
@@ -33,8 +33,9 @@ public class DataTableInsertColumnHeaderCommand extends DesignerCommand
     private Element _dataTable;
 
     /**
-     * @param label
      * @param viewer
+     * @param dataTable 
+     * @param header 
      */
     public DataTableInsertColumnHeaderCommand(IHTMLGraphicalViewer viewer, Element dataTable, boolean header)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertHeaderCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertHeaderCommand.java
index 10f0c40b0..4231efc1a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertHeaderCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/DataTableInsertHeaderCommand.java
@@ -33,8 +33,9 @@ public class DataTableInsertHeaderCommand extends DesignerCommand
     private Element _dataTable;
 
     /**
-     * @param label
      * @param viewer
+     * @param dataTable 
+     * @param header 
      */
     public DataTableInsertHeaderCommand(IHTMLGraphicalViewer viewer, Element dataTable, boolean header)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteColumnCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteColumnCommand.java
index 46116d396..a57f4f763 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteColumnCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteColumnCommand.java
@@ -14,12 +14,14 @@ package org.eclipse.jst.pagedesigner.jsf.ui.commands.jsfhtml;
 import java.util.ArrayList;
 import java.util.List;
 
+import org.eclipse.core.runtime.IStatus;
 import org.eclipse.jface.viewers.ISelection;
 import org.w3c.dom.Element;
 import org.w3c.dom.Node;
 
 import org.eclipse.jst.pagedesigner.commands.DesignerCommand;
 import org.eclipse.jst.pagedesigner.jsf.core.dom.JSFDOMUtil;
+import org.eclipse.jst.pagedesigner.jsf.ui.JSFUIPlugin;
 import org.eclipse.jst.pagedesigner.utils.DOMUtil;
 import org.eclipse.jst.pagedesigner.viewer.IHTMLGraphicalViewer;
 
@@ -36,8 +38,9 @@ public class PanelGridDeleteColumnCommand extends DesignerCommand
     private Element _panelGridEle;
 
     /**
-     * @param label
      * @param viewer
+     * @param panelGrid 
+     * @param columnIndex 
      */
     public PanelGridDeleteColumnCommand(IHTMLGraphicalViewer viewer, Element panelGrid, int columnIndex)
     {
@@ -77,8 +80,7 @@ public class PanelGridDeleteColumnCommand extends DesignerCommand
 
         if (_columnIndex >= columns)
         {
-            // should not happen
-            // TODO: log.
+            JSFUIPlugin.log(IStatus.ERROR, "Should not happen", new Throwable());
             return;
         }
         List children = JSFDOMUtil.getUIComponentChildren(_panelGridEle);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteRowCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteRowCommand.java
index 71ccabf81..e0a27953c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteRowCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridDeleteRowCommand.java
@@ -35,8 +35,9 @@ public class PanelGridDeleteRowCommand extends DesignerCommand
     int     _rowIndex;
 
     /**
-     * @param label
      * @param viewer
+     * @param panelGrid 
+     * @param index 
      */
     public PanelGridDeleteRowCommand(IHTMLGraphicalViewer viewer, Element panelGrid, int index)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertFooterCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertFooterCommand.java
index f4857baac..d3cb10952 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertFooterCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertFooterCommand.java
@@ -30,6 +30,10 @@ public class PanelGridInsertFooterCommand extends DesignerCommand
 {
     private Element _panelGrid;
 
+    /**
+     * @param viewer
+     * @param panelGrid
+     */
     public PanelGridInsertFooterCommand(IHTMLGraphicalViewer viewer, Element panelGrid)
     {
         super(CommandResources.getString("PanelGridInsertFooterCommand.Label.InsertFooter"), viewer); //$NON-NLS-1$
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertHeaderCommand.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertHeaderCommand.java
index 8d47e54c0..6a9753db5 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertHeaderCommand.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/commands/jsfhtml/PanelGridInsertHeaderCommand.java
@@ -28,6 +28,10 @@ public class PanelGridInsertHeaderCommand extends DesignerCommand
 {
     private Element _panelGrid;
 
+    /**
+     * @param viewer
+     * @param panelGrid
+     */
     public PanelGridInsertHeaderCommand(IHTMLGraphicalViewer viewer, Element panelGrid)
     {
         super(CommandResources.getString("PanelGridInsertHeaderCommand.Label.InsertHeader"), viewer); //$NON-NLS-1$
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfcore/LoadBundleTagConverter.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfcore/LoadBundleTagConverter.java
index a354a47ff..760808065 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfcore/LoadBundleTagConverter.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfcore/LoadBundleTagConverter.java
@@ -44,7 +44,7 @@ public class LoadBundleTagConverter extends HiddenTagConverter
 
     /**
      * @param host
-     * @param image
+     * @param labelProvider 
      */
     public LoadBundleTagConverter(Element host, ILabelProvider labelProvider)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/DataTableTagConverter.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/DataTableTagConverter.java
index d788ab100..dba3f4420 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/DataTableTagConverter.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/DataTableTagConverter.java
@@ -15,7 +15,9 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.StringTokenizer;
 
+import org.eclipse.jst.jsf.common.internal.provisional.dom.TagIdentifier;
 import org.eclipse.jst.jsf.core.internal.tld.IJSFConstants;
+import org.eclipse.jst.jsf.core.internal.tld.TagIdentifierFactory;
 import org.eclipse.jst.pagedesigner.IHTMLConstants;
 import org.eclipse.jst.pagedesigner.converter.AbstractTagConverter;
 import org.eclipse.jst.pagedesigner.converter.ConvertPosition;
@@ -94,8 +96,8 @@ public class DataTableTagConverter extends AbstractTagConverter
                 if (child instanceof Element)
                 {
                     Element ele = (Element) child;
-                    // XXX: we are not handling namespace here
-                    if (IJSFConstants.TAG_COLUMN.equals(ele.getLocalName()))
+                    TagIdentifier  tagId = TagIdentifierFactory.createDocumentTagWrapper(ele);
+                    if (IJSFConstants.TAG_IDENTIFIER_COLUMN.isSameTagType(tagId))
                     {
                         result.add(ele);
                     }
@@ -106,6 +108,11 @@ public class DataTableTagConverter extends AbstractTagConverter
         return result;
     }
 
+    /**
+     * @param hostEle
+     * @param tableEle
+     * @param columns
+     */
     protected void convertTBody(Element hostEle, Element tableEle, List columns)
     {
         // Rendering the table body
@@ -172,6 +179,7 @@ public class DataTableTagConverter extends AbstractTagConverter
     /**
      * @param hostEle
      * @param tableEle
+     * @param columns 
      * @param header true means header, false means footer
      */
     protected void convertTHeader(Element hostEle, Element tableEle, List columns, boolean header)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/InputTagConverter.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/InputTagConverter.java
index 9262eedf0..865e9aa73 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/InputTagConverter.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/InputTagConverter.java
@@ -30,6 +30,7 @@ public class InputTagConverter extends AbstractTagConverter
 
     /**
      * @param host
+     * @param inputType 
      */
     public InputTagConverter(Element host, String inputType)
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectItemModel.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectItemModel.java
index b10a977b4..089c92f5b 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectItemModel.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectItemModel.java
@@ -44,6 +44,7 @@ public class SelectItemModel
     
     /**
      * Return a description of this item, for use in development tools.
+     * @return the description
      */ 
     public String 	getDescription()
     {
@@ -52,7 +53,7 @@ public class SelectItemModel
 
     /**
      * Return the label of this item, to be rendered visibly for the user.
-     * @return
+     * @return the label for this item
      */
     public String 	getLabel()
     {
@@ -76,36 +77,57 @@ public class SelectItemModel
     {
         this._label = label;
     }
+    /**
+     * @return the item value
+     */
     public String getItemValue()
     {
         return _itemValue;
     }
 
+    /**
+     * @param itemValue
+     */
     public void setItemValue(String itemValue)
     {
         this._itemValue = itemValue;
     }
 
+    /**
+     * @return the value
+     */
     public String getValue()
     {
         return _value;
     }
 
+    /**
+     * @param value
+     */
     public void setValue(String value)
     {
         this._value = value;
     }
 
+    /**
+     * @return the id
+     */
     public String getId()
     {
         return _id;
     }
 
+    /**
+     * @param id
+     */
     public void setId(String id)
     {
         _id = id;
     }
 
+    /**
+     * @return the display string
+     */
     public String getDisplayString()
     {
         if (_label != null && _label.length() > 0)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectTagConverter.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectTagConverter.java
index fab20356a..0beeac4c2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectTagConverter.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/SelectTagConverter.java
@@ -40,7 +40,7 @@ public abstract class SelectTagConverter extends AbstractTagConverter
      * Return a list of SelectItem
      * 
      * @param parent
-     * @return
+     * @return the select item list for parent
      */
     protected List getSelectItems(Element parent)
     {
@@ -88,6 +88,10 @@ public abstract class SelectTagConverter extends AbstractTagConverter
         return result;
     }
 
+    /**
+     * @param parent
+     * @return the select item model for parent
+     */
     public SelectItemModel getDefault(Element parent)
     {
         SelectItemModel item = new SelectItemModel();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/TableBasedSelectTagConverter.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/TableBasedSelectTagConverter.java
index 5a2a38604..043b82d3b 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/TableBasedSelectTagConverter.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/jsfhtml/TableBasedSelectTagConverter.java
@@ -30,6 +30,7 @@ public class TableBasedSelectTagConverter extends SelectTagConverter
 
     /**
      * @param host
+     * @param inputType 
      */
     public TableBasedSelectTagConverter(Element host, String inputType)
     {
@@ -158,6 +159,9 @@ public class TableBasedSelectTagConverter extends SelectTagConverter
         return (tableEle);
     }
 
+    /**
+     * @return the input type
+     */
     protected String getInputType()
     {
         return _inputType;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/operations/jsf/LoadBundleOperation.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/operations/jsf/LoadBundleOperation.java
index ff91a7bb2..aabe03066 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/operations/jsf/LoadBundleOperation.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/converter/operations/jsf/LoadBundleOperation.java
@@ -121,7 +121,7 @@ public class LoadBundleOperation extends AbstractTransformOperation {
 	 * 
 	 * @param project IProject instance.
 	 * @param filePath Source folder-relative path of the file to be located.
-	 * @return
+	 * @return the specified file in any source folder of the specified project.
 	 */
 	protected IResource findFileInSrcFolder(IProject project, String filePath) {
 		IResource resource = null;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/FacetElementEdit.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/FacetElementEdit.java
index 300fc7b19..cf5ac89af 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/FacetElementEdit.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/FacetElementEdit.java
@@ -16,6 +16,12 @@ import org.eclipse.jst.pagedesigner.elementedit.AbstractElementEdit;
 import org.eclipse.jst.pagedesigner.parts.ElementEditPart;
 import org.w3c.dom.Element;
 
+/**
+ * Element edit for a f:facet
+ * 
+ * @author cbateman
+ *
+ */
 public class FacetElementEdit extends AbstractElementEdit
 {
     /* (non-Javadoc)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/JarEntryEditorInput.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/JarEntryEditorInput.java
index 34595153b..f6b5697ab 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/JarEntryEditorInput.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/JarEntryEditorInput.java
@@ -27,6 +27,9 @@ public class JarEntryEditorInput implements IStorageEditorInput
 
     private IStorage fJarEntryFile;
 
+    /**
+     * @param jarEntryFile
+     */
     public JarEntryEditorInput(IStorage jarEntryFile)
     {
         fJarEntryFile = jarEntryFile;
@@ -64,16 +67,18 @@ public class JarEntryEditorInput implements IStorageEditorInput
         return fJarEntryFile.getName();
     }
 
-    /*
-     * @see IEditorInput#getFullPath()
+
+    /**
+     * @return the full path of the entry as a string
      */
     public String getFullPath()
     {
         return fJarEntryFile.getFullPath().toString();
     }
 
-    /*
-     * @see IEditorInput#getContentType()
+
+    /**
+     * @return the file extension of this input
      */
     public String getContentType()
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/LoadBundleAction.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/LoadBundleAction.java
index 2c784467a..8e03cb1c4 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/LoadBundleAction.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfcore/LoadBundleAction.java
@@ -37,6 +37,9 @@ public class LoadBundleAction extends Action
     private String   _baseName;
     private IProject _project;
 
+    /**
+     * Create a new action
+     */
     public LoadBundleAction()
     {
         setText(JSFUIPlugin.getResourceString("ElementEdit.Submenu.LoadBundle"));//$NON-NLS-1$
@@ -112,6 +115,11 @@ public class LoadBundleAction extends Action
         this._project = projectFor;
     }
 
+    /**
+     * @param inputName
+     * @return the editor id of theditor to open for the inputName or null
+     * if none can be ascertained
+     */
     public String getEditorID(String inputName)
     {
         IEditorRegistry registry = PlatformUI.getWorkbench().getEditorRegistry();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/ColumnElementEdit.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/ColumnElementEdit.java
index d162857fe..61f0c0f9e 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/ColumnElementEdit.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/ColumnElementEdit.java
@@ -68,6 +68,9 @@ public class ColumnElementEdit extends DefaultJSFHTMLElementEdit
 
     private static class MyDropLocationStrategy extends DefaultDropLocationStrategy
     {
+        /**
+         * @param viewer
+         */
         public MyDropLocationStrategy(EditPartViewer viewer) {
             super(viewer);
         }
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DataTableElementEdit.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DataTableElementEdit.java
index f5b4d5b0a..515796247 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DataTableElementEdit.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DataTableElementEdit.java
@@ -235,6 +235,9 @@ public class DataTableElementEdit extends DefaultJSFHTMLElementEdit
         ElementEditPart _part;
         Cursor          _columnSelectCursor;
 
+        /**
+         * @param part
+         */
         public DataTableResizePolicy(ElementEditPart part)
         {
             _part = part;
@@ -268,7 +271,7 @@ public class DataTableElementEdit extends DefaultJSFHTMLElementEdit
             Element dataTable = (Element) _part.getIDOMNode();
             if (request instanceof TableResizeRequest)
             {
-                // TODO: FIXME:
+                return super.getCommand(request);
             }
             else if (request instanceof TableInsertRequest)
             {
@@ -362,6 +365,10 @@ public class DataTableElementEdit extends DefaultJSFHTMLElementEdit
             return getSelectionTracker(request);
         }
 
+        /**
+         * @param mouseLocation
+         * @return the edit part fo the given mouse location
+         */
         public EditPart getRetargetSelectionEditPart(Point mouseLocation) {
             if (hitTestColumnSelection(mouseLocation))
             {
@@ -415,6 +422,10 @@ public class DataTableElementEdit extends DefaultJSFHTMLElementEdit
         }
     }
     
+    /**
+     * @author cbateman
+     *
+     */
     public static class MyDragMoveEditPolicy extends DragMoveEditPolicy 
     {
         protected IPositionMediator createDropChildValidator(
@@ -435,6 +446,10 @@ public class DataTableElementEdit extends DefaultJSFHTMLElementEdit
         
         private static class OnlyColumnsAndFacetsRule extends DefaultPositionRule
         {
+            /**
+             * @param mediator
+             * @param actionData
+             */
             public OnlyColumnsAndFacetsRule(IPositionMediator mediator,
                     ActionData actionData) {
                 super(mediator, actionData);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DefaultJSFHTMLElementEdit.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DefaultJSFHTMLElementEdit.java
index cb76b24e8..2bfcd06cb 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DefaultJSFHTMLElementEdit.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/DefaultJSFHTMLElementEdit.java
@@ -42,6 +42,9 @@ public class DefaultJSFHTMLElementEdit extends AbstractElementEdit
     {
         String _localName;
 
+        /**
+         * @param localtag
+         */
         public JSFCoreSupport(String localtag)
         {
             _localName = localtag;
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/PanelGridElementEdit.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/PanelGridElementEdit.java
index 08c210e4d..671960791 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/PanelGridElementEdit.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/jsfhtml/PanelGridElementEdit.java
@@ -235,6 +235,9 @@ public class PanelGridElementEdit extends DefaultJSFHTMLElementEdit
     {
         ElementEditPart _part;
 
+        /**
+         * @param part
+         */
         public PanelGridResizePolicy(ElementEditPart part)
         {
             _part = part;
@@ -275,7 +278,7 @@ public class PanelGridElementEdit extends DefaultJSFHTMLElementEdit
             Element panelGrid = (Element) _part.getIDOMNode();
             if (request instanceof TableResizeRequest)
             {
-                // TODO: FIXME:
+                return super.getCommand(request);
             }
             else if (request instanceof TableInsertRequest)
             {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHColumnHeaderFooterRequest.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHColumnHeaderFooterRequest.java
index 6725b3246..076fd74ad 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHColumnHeaderFooterRequest.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHColumnHeaderFooterRequest.java
@@ -21,6 +21,10 @@ public class DeleteHColumnHeaderFooterRequest extends Request
 {
     private boolean _isHeader;
 
+    /**
+     * @param type
+     * @param isHeader
+     */
     public DeleteHColumnHeaderFooterRequest(String type, boolean isHeader)
     {
         super(type);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHeaderFooterRequest.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHeaderFooterRequest.java
index cd67259ce..d935652bb 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHeaderFooterRequest.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/DeleteHeaderFooterRequest.java
@@ -21,6 +21,10 @@ public class DeleteHeaderFooterRequest extends Request
 {
     private boolean _isHeader;
 
+    /**
+     * @param type
+     * @param isHeader
+     */
     public DeleteHeaderFooterRequest(String type, boolean isHeader)
     {
         super(type);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHColumnHeaderFooterRequest.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHColumnHeaderFooterRequest.java
index 9ed09a9f0..34e3d4a69 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHColumnHeaderFooterRequest.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHColumnHeaderFooterRequest.java
@@ -21,6 +21,10 @@ public class InsertHColumnHeaderFooterRequest extends Request
 {
     private boolean _isHeader;
 
+    /**
+     * @param type
+     * @param isHeader
+     */
     public InsertHColumnHeaderFooterRequest(String type, boolean isHeader)
     {
         super(type);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHeaderFooterRequest.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHeaderFooterRequest.java
index 404ac275e..31e853f78 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHeaderFooterRequest.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/request/InsertHeaderFooterRequest.java
@@ -21,6 +21,10 @@ public class InsertHeaderFooterRequest extends Request
 {
     private boolean _isHeader;
 
+    /**
+     * @param type
+     * @param isHeader
+     */
     public InsertHeaderFooterRequest(String type, boolean isHeader)
     {
         super(type);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/DataTableUtil.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/DataTableUtil.java
index c65148a3a..a6f36026b 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/DataTableUtil.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/DataTableUtil.java
@@ -27,11 +27,18 @@ public class DataTableUtil
     private static final int INVALID_POSITION = -10;
     private Element          _dataTable       = null;
 
+    /**
+     * @param dataTable
+     */
     public DataTableUtil(Element dataTable)
     {
         this._dataTable = dataTable;
     }
 
+    /**
+     * @param column
+     * @return the column index for column
+     */
     public int getColumnIndex(Element column)
     {
         NodeList nodeList = this._dataTable.getElementsByTagName("h:column");
@@ -48,6 +55,10 @@ public class DataTableUtil
         return INVALID_POSITION;
     }
 
+    /**
+     * @param child
+     * @return the hcolumn parent of child or null if not found
+     */
     public Element findHColumnParent(Node child)
     {
         if (child == null)
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/PanelGridUtil.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/PanelGridUtil.java
index e04492b22..bc57d2840 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/PanelGridUtil.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/elementedit/util/PanelGridUtil.java
@@ -28,11 +28,18 @@ public class PanelGridUtil
     private static final int INVALID_POSITION = -10;
     private Element          _panelGrid       = null;
 
+    /**
+     * @param panelGrid
+     */
     public PanelGridUtil(Element panelGrid)
     {
         this._panelGrid = panelGrid;
     }
 
+    /**
+     * @param domIndex
+     * @return the row index in the panel of the relative dom index
+     */
     public int convertRowIndexFromDomToView(int domIndex)
     {
         boolean hasHeaderRow = (JSFDOMUtil.findFacet(this._panelGrid, "header") != null);
@@ -64,6 +71,10 @@ public class PanelGridUtil
         return domIndex + uiRows;
     }
 
+    /**
+     * @param cell
+     * @return the dom row index of cell in the panel
+     */
     public int getDomRowIndex(Element cell)
     {
         boolean hasHeaderRow = (JSFDOMUtil.findFacet(this._panelGrid, "header") != null);
@@ -113,6 +124,10 @@ public class PanelGridUtil
         return rowIndex;
     }
 
+    /**
+     * @param cell
+     * @return the column index of cell in the panel
+     */
     public int getDomColumnIndex(Element cell)
     {
         int columns = DOMUtil.getIntAttributeIgnoreCase(this._panelGrid, IJSFConstants.ATTR_COLUMNS, 1);
@@ -131,6 +146,9 @@ public class PanelGridUtil
         return columnIndex;
     }
 
+    /**
+     * @return the row count in the panel
+     */
     public int getUIRowCount()
     {
         int columns = DOMUtil.getIntAttributeIgnoreCase(this._panelGrid, IJSFConstants.ATTR_COLUMNS, 1);
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/IInputWidgetTypes.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/IInputWidgetTypes.java
new file mode 100644
index 000000000..b131a7a9d
--- /dev/null
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/IInputWidgetTypes.java
@@ -0,0 +1,22 @@
+package org.eclipse.jst.pagedesigner.jsf.ui.sections;
+
+interface IInputWidgetTypes 
+{
+    /**
+     *  a hidden input
+     */
+    public static final int       HIDDEN   = 0;
+    /**
+     * a secret input
+     */
+    public static final int       SECRET   = 1;
+    /**
+     * an input box
+     */
+    public static final int       TEXT     = 2;
+    /**
+     * an input text area
+     */
+    public static final int       TEXTAREA = 3;
+
+}
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreActionListenerSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreActionListenerSection.java
index 9a6f6c9e1..09958087f 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreActionListenerSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreActionListenerSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreActionListenerSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreActionListenerSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_ACTIONLISTENER, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreAttributeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreAttributeSection.java
index a69146de7..2767db4e2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreAttributeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreAttributeSection.java
@@ -19,7 +19,10 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  * @author mengbo
  */
 public class JSFCoreAttributeSection extends AttributeGroupSection
-{
+{    
+    /**
+     * The default constructor
+     */
     public JSFCoreAttributeSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_ATTRIBUTE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertDateTimeGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertDateTimeGroup.java
index d55345a32..891975f04 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertDateTimeGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertDateTimeGroup.java
@@ -57,9 +57,7 @@ public class JSFCoreConvertDateTimeGroup extends AttributeGroup
                                                    };
 
     /**
-     * @param uri
-     * @param tagName
-     * @param attrNames
+     * The default constructor
      */
     public JSFCoreConvertDateTimeGroup()
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertNumberGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertNumberGroup.java
index 53c2172bf..760847bcb 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertNumberGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConvertNumberGroup.java
@@ -42,9 +42,7 @@ public class JSFCoreConvertNumberGroup extends AttributeGroup
                                         };
 
     /**
-     * @param uri
-     * @param tagName
-     * @param attrNames
+     * The default constructor
      */
     public JSFCoreConvertNumberGroup()
     {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConverterSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConverterSection.java
index f5ad09c22..d4eac787e 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConverterSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreConverterSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreConverterSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreConverterSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_CONVERTER, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreFacetSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreFacetSection.java
index 3bf1d5d76..cb840d118 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreFacetSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreFacetSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreFacetSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreFacetSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_FACET, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreLoadBundleSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreLoadBundleSection.java
index a91bf7c75..769060cda 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreLoadBundleSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreLoadBundleSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreLoadBundleSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreLoadBundleSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_LOADBUNDLE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreParamSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreParamSection.java
index 3accb6db6..5794d9206 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreParamSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreParamSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreParamSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreParamSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_PARAM, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemSection.java
index 10b37f718..e5801c98e 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreSelectItemSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreSelectItemSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_SELECTITEM, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemsSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemsSection.java
index ceed8c815..a2390c24c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemsSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSelectItemsSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreSelectItemsSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreSelectItemsSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_SELECTITEMS, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSubviewSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSubviewSection.java
index 8e55322ee..af1880ea6 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSubviewSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreSubviewSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreSubviewSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreSubviewSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_SUBVIEW, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateDoubleRangeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateDoubleRangeSection.java
index e939f99ee..747a3433e 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateDoubleRangeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateDoubleRangeSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreValidateDoubleRangeSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreValidateDoubleRangeSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VALIDATEDOUBLERANGE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLengthSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLengthSection.java
index d2d503b3c..6c60c9ded 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLengthSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLengthSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreValidateLengthSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreValidateLengthSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VALIDATELENGTH, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLongRangeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLongRangeSection.java
index 5c8c7d95e..7933d9cc8 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLongRangeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidateLongRangeSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreValidateLongRangeSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreValidateLongRangeSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VALIDATELONGRANGE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidatorSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidatorSection.java
index 53ea05724..aa9d9bfa7 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidatorSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValidatorSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreValidatorSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreValidatorSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VALIDATOR, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValueChangeListenerSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValueChangeListenerSection.java
index 61d23ff22..0db034474 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValueChangeListenerSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreValueChangeListenerSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreValueChangeListenerSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreValueChangeListenerSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VALUECHANGELISTENER, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreVerbatimSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreVerbatimSection.java
index 410e7b83f..ec0eea565 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreVerbatimSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreVerbatimSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreVerbatimSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreVerbatimSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VERBATIM, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreViewSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreViewSection.java
index 4f17617dc..d08a2f71a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreViewSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFCoreViewSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFCoreViewSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFCoreViewSection()
     {
         super(ITLDConstants.URI_JSF_CORE, IJSFConstants.TAG_VIEW, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlColumnSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlColumnSection.java
index 73e17809e..6791f4664 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlColumnSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlColumnSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlColumnSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlColumnSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_COLUMN, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandButtonSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandButtonSection.java
index d7b1ef050..9abaf11fc 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandButtonSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandButtonSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlCommandButtonSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlCommandButtonSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_COMMANDBUTTON, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandLinkSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandLinkSection.java
index 2d6900577..320c4bfc8 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandLinkSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlCommandLinkSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlCommandLinkSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlCommandLinkSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_COMMANDLINK, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlDataTableSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlDataTableSection.java
index bca360327..89bbeb3a9 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlDataTableSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlDataTableSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlDataTableSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlDataTableSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_DATATABLE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlFormSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlFormSection.java
index 13ccce05e..1fe391dae 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlFormSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlFormSection.java
@@ -20,7 +20,10 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlFormSection extends AttributeGroupSection
 {
-    public JSFHtmlFormSection()
+    /**
+     * The default constructor
+     */
+   public JSFHtmlFormSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_FORM, 
                 new String[] {
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlGraphicImageSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlGraphicImageSection.java
index 751f86440..3475f7bef 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlGraphicImageSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlGraphicImageSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlGraphicImageSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlGraphicImageSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_GRAPHICIMAGE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenSection.java
index 96e72a976..04c47b210 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlInputHiddenSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlInputHiddenSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_INPUTHIDDEN, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenTypeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenTypeSection.java
index 5e6d7f677..e5bba5fd9 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenTypeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputHiddenTypeSection.java
@@ -37,16 +37,15 @@ public class JSFHtmlInputHiddenTypeSection extends BaseCustomSection
 {
     private RadiosDialogField     _typesField;
 
-    public static final int       HIDDEN   = 0;
-    public static final int       SECRET   = 1;
-    public static final int       TEXT     = 2;
-    public static final int       TEXTAREA = 3;
     final private static String[] TYPES    = { SectionResources.getString("JSFHtmlInputHiddenSection.Type0"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type1"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type2"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type3") //$NON-NLS-1$
                                            };
 
+    /**
+     * The default constructor
+     */
     public JSFHtmlInputHiddenTypeSection()
     {
         super();
@@ -62,7 +61,7 @@ public class JSFHtmlInputHiddenTypeSection extends BaseCustomSection
                 String localTag = toLocalTag(type);
                 String newtag = prefix + ":" + localTag; //$NON-NLS-1$
                 Map attrs = new HashMap();
-                if (type == TEXTAREA)
+                if (type == IInputWidgetTypes.TEXTAREA)
                 {
                     // need to change "size" attribute to "cols"
                     String size = _element.getAttribute(IJSFConstants.ATTR_SIZE);
@@ -108,13 +107,13 @@ public class JSFHtmlInputHiddenTypeSection extends BaseCustomSection
     {
         switch (type)
         {
-            case SECRET:
+            case IInputWidgetTypes.SECRET:
                 return IJSFConstants.TAG_INPUTSECRET;
-            case HIDDEN:
+            case IInputWidgetTypes.HIDDEN:
                 return IJSFConstants.TAG_INPUTHIDDEN;
-            case TEXTAREA:
+            case IInputWidgetTypes.TEXTAREA:
                 return IJSFConstants.TAG_INPUTTEXTAREA;
-            case TEXT:
+            case IInputWidgetTypes.TEXT:
                 return IJSFConstants.TAG_INPUTTEXT;
             default:
                 return IJSFConstants.TAG_INPUTHIDDEN;
@@ -128,28 +127,31 @@ public class JSFHtmlInputHiddenTypeSection extends BaseCustomSection
         _typesField.setSelectedIndexWithoutUpdate(getInputType());
     }
 
+    /**
+     * @return the type of the input.  One of TEXT, TEXTAREA, HIDDEN, SECRET.
+     */
     public int getInputType()
     {
         String localTag = _element.getLocalName();
         if (IJSFConstants.TAG_INPUTTEXT.equals(localTag))
         {
-            return TEXT;
+            return IInputWidgetTypes.TEXT;
         }
         else if (IJSFConstants.TAG_INPUTSECRET.equals(localTag))
         {
-            return SECRET;
+            return IInputWidgetTypes.SECRET;
         }
         else if (IJSFConstants.TAG_INPUTHIDDEN.equals(localTag))
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
         else if (IJSFConstants.TAG_INPUTTEXTAREA.equals(localTag))
         {
-            return TEXTAREA;
+            return IInputWidgetTypes.TEXTAREA;
         }
         else
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
     }
 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputOthersSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputOthersSection.java
index d364f5472..55a7efa8b 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputOthersSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputOthersSection.java
@@ -470,6 +470,9 @@ public class JSFHtmlInputOthersSection extends BaseCustomSection
         }
     }
 
+    /**
+     * The default constructor
+     */
     public JSFHtmlInputOthersSection()
     {
         super();
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretSection.java
index 450b11766..7f7b212f2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlInputSecretSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor
+     */
     public JSFHtmlInputSecretSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_INPUTSECRET, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretTypeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretTypeSection.java
index 0188ad7ed..e1eda9f8a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretTypeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputSecretTypeSection.java
@@ -37,16 +37,15 @@ public class JSFHtmlInputSecretTypeSection extends BaseCustomSection
 {
     private RadiosDialogField     _typesField;
 
-    public static final int       HIDDEN   = 0;
-    public static final int       SECRET   = 1;
-    public static final int       TEXT     = 2;
-    public static final int       TEXTAREA = 3;
     final private static String[] TYPES    = { SectionResources.getString("JSFHtmlInputHiddenSection.Type0"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type1"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type2"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type3") //$NON-NLS-1$
                                            };
 
+    /**
+     * The default constructor
+     */
     public JSFHtmlInputSecretTypeSection()
     {
         super();
@@ -62,7 +61,7 @@ public class JSFHtmlInputSecretTypeSection extends BaseCustomSection
                 String localTag = toLocalTag(type);
                 String newtag = prefix + ":" + localTag; //$NON-NLS-1$
                 Map attrs = new HashMap();
-                if (type == TEXTAREA)
+                if (type == IInputWidgetTypes.TEXTAREA)
                 {
                     // need to change "size" attribute to "cols"
                     String size = _element.getAttribute(IJSFConstants.ATTR_SIZE);
@@ -108,13 +107,13 @@ public class JSFHtmlInputSecretTypeSection extends BaseCustomSection
     {
         switch (type)
         {
-            case SECRET:
+            case IInputWidgetTypes.SECRET:
                 return IJSFConstants.TAG_INPUTSECRET;
-            case HIDDEN:
+            case IInputWidgetTypes.HIDDEN:
                 return IJSFConstants.TAG_INPUTHIDDEN;
-            case TEXTAREA:
+            case IInputWidgetTypes.TEXTAREA:
                 return IJSFConstants.TAG_INPUTTEXTAREA;
-            case TEXT:
+            case IInputWidgetTypes.TEXT:
                 return IJSFConstants.TAG_INPUTTEXT;
             default:
                 return IJSFConstants.TAG_INPUTHIDDEN;
@@ -127,28 +126,31 @@ public class JSFHtmlInputSecretTypeSection extends BaseCustomSection
         _typesField.setSelectedIndexWithoutUpdate(getInputType());
     }
 
+    /**
+     * @return the input type
+     */
     public int getInputType()
     {
         String localTag = _element.getLocalName();
         if (IJSFConstants.TAG_INPUTTEXT.equals(localTag))
         {
-            return TEXT;
+            return IInputWidgetTypes.TEXT;
         }
         else if (IJSFConstants.TAG_INPUTSECRET.equals(localTag))
         {
-            return SECRET;
+            return IInputWidgetTypes.SECRET;
         }
         else if (IJSFConstants.TAG_INPUTHIDDEN.equals(localTag))
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
         else if (IJSFConstants.TAG_INPUTTEXTAREA.equals(localTag))
         {
-            return TEXTAREA;
+            return IInputWidgetTypes.TEXTAREA;
         }
         else
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
     }
 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextSection.java
index d8750bae6..2360ee255 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlInputTextSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlInputTextSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_INPUTTEXT, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextTypeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextTypeSection.java
index 32637bbc2..d757e0778 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextTypeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextTypeSection.java
@@ -37,16 +37,15 @@ public class JSFHtmlInputTextTypeSection extends BaseCustomSection
 {
     private RadiosDialogField     _typesField;
 
-    public static final int       HIDDEN   = 0;
-    public static final int       SECRET   = 1;
-    public static final int       TEXT     = 2;
-    public static final int       TEXTAREA = 3;
     final private static String[] TYPES    = { SectionResources.getString("JSFHtmlInputHiddenSection.Type0"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type1"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type2"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type3") //$NON-NLS-1$
                                            };
 
+    /**
+     * The default constructor 
+     */
     public JSFHtmlInputTextTypeSection()
     {
         super();
@@ -62,7 +61,7 @@ public class JSFHtmlInputTextTypeSection extends BaseCustomSection
                 String localTag = toLocalTag(type);
                 String newtag = prefix + ":" + localTag; //$NON-NLS-1$
                 Map attrs = new HashMap();
-                if (type == TEXTAREA)
+                if (type == IInputWidgetTypes.TEXTAREA)
                 {
                     // need to change "size" attribute to "cols"
                     String size = _element.getAttribute(IJSFConstants.ATTR_SIZE);
@@ -108,13 +107,13 @@ public class JSFHtmlInputTextTypeSection extends BaseCustomSection
     {
         switch (type)
         {
-            case SECRET:
+            case IInputWidgetTypes.SECRET:
                 return IJSFConstants.TAG_INPUTSECRET;
-            case HIDDEN:
+            case IInputWidgetTypes.HIDDEN:
                 return IJSFConstants.TAG_INPUTHIDDEN;
-            case TEXTAREA:
+            case IInputWidgetTypes.TEXTAREA:
                 return IJSFConstants.TAG_INPUTTEXTAREA;
-            case TEXT:
+            case IInputWidgetTypes.TEXT:
                 return IJSFConstants.TAG_INPUTTEXT;
             default:
                 return IJSFConstants.TAG_INPUTHIDDEN;
@@ -128,28 +127,31 @@ public class JSFHtmlInputTextTypeSection extends BaseCustomSection
         _typesField.setSelectedIndexWithoutUpdate(getInputType());
     }
 
+    /**
+     * @return the input type. One of IInputWidgetTypes.
+     */
     public int getInputType()
     {
         String localTag = _element.getLocalName();
         if (IJSFConstants.TAG_INPUTTEXT.equals(localTag))
         {
-            return TEXT;
+            return IInputWidgetTypes.TEXT;
         }
         else if (IJSFConstants.TAG_INPUTSECRET.equals(localTag))
         {
-            return SECRET;
+            return IInputWidgetTypes.SECRET;
         }
         else if (IJSFConstants.TAG_INPUTHIDDEN.equals(localTag))
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
         else if (IJSFConstants.TAG_INPUTTEXTAREA.equals(localTag))
         {
-            return TEXTAREA;
+            return IInputWidgetTypes.TEXTAREA;
         }
         else
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
     }
 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaSection.java
index 365004043..bb4f5c902 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlInputTextareaSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlInputTextareaSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_INPUTTEXTAREA, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaTypeSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaTypeSection.java
index 93731de14..dab7f9711 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaTypeSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlInputTextareaTypeSection.java
@@ -30,6 +30,7 @@ import org.eclipse.wst.common.ui.properties.internal.provisional.TabbedPropertyS
 import org.eclipse.wst.sse.core.internal.provisional.INodeNotifier;
 
 /**
+ * FIXME: there is a lot of copied code for this input sections
  * @author mengbo
  * @version 1.5
  */
@@ -37,16 +38,15 @@ public class JSFHtmlInputTextareaTypeSection extends BaseCustomSection
 {
     private RadiosDialogField     _typesField;
 
-    public static final int       HIDDEN   = 0;
-    public static final int       SECRET   = 1;
-    public static final int       TEXT     = 2;
-    public static final int       TEXTAREA = 3;
     final private static String[] TYPES    = { SectionResources.getString("JSFHtmlInputHiddenSection.Type0"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type1"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type2"), //$NON-NLS-1$
                                            SectionResources.getString("JSFHtmlInputHiddenSection.Type3") //$NON-NLS-1$
                                            };
 
+    /**
+     * The default constructor 
+     */
     public JSFHtmlInputTextareaTypeSection()
     {
         super();
@@ -62,7 +62,7 @@ public class JSFHtmlInputTextareaTypeSection extends BaseCustomSection
                 String localTag = toLocalTag(type);
                 String newtag = prefix + ":" + localTag; //$NON-NLS-1$
                 Map attrs = new HashMap();
-                if (type == TEXTAREA)
+                if (type == IInputWidgetTypes.TEXTAREA)
                 {
                     // need to change "size" attribute to "cols"
                     String size = _element.getAttribute(IJSFConstants.ATTR_SIZE);
@@ -108,13 +108,13 @@ public class JSFHtmlInputTextareaTypeSection extends BaseCustomSection
     {
         switch (type)
         {
-            case SECRET:
+            case IInputWidgetTypes.SECRET:
                 return IJSFConstants.TAG_INPUTSECRET;
-            case HIDDEN:
+            case IInputWidgetTypes.HIDDEN:
                 return IJSFConstants.TAG_INPUTHIDDEN;
-            case TEXTAREA:
+            case IInputWidgetTypes.TEXTAREA:
                 return IJSFConstants.TAG_INPUTTEXTAREA;
-            case TEXT:
+            case IInputWidgetTypes.TEXT:
                 return IJSFConstants.TAG_INPUTTEXT;
             default:
                 return IJSFConstants.TAG_INPUTHIDDEN;
@@ -128,28 +128,31 @@ public class JSFHtmlInputTextareaTypeSection extends BaseCustomSection
         _typesField.setSelectedIndexWithoutUpdate(getInputType());
     }
 
+    /**
+     * @return one of IInputWidgetTypes
+     */
     public int getInputType()
     {
         String localTag = _element.getLocalName();
         if (IJSFConstants.TAG_INPUTTEXT.equals(localTag))
         {
-            return TEXT;
+            return IInputWidgetTypes.TEXT;
         }
         else if (IJSFConstants.TAG_INPUTSECRET.equals(localTag))
         {
-            return SECRET;
+            return IInputWidgetTypes.SECRET;
         }
         else if (IJSFConstants.TAG_INPUTHIDDEN.equals(localTag))
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
         else if (IJSFConstants.TAG_INPUTTEXTAREA.equals(localTag))
         {
-            return TEXTAREA;
+            return IInputWidgetTypes.TEXTAREA;
         }
         else
         {
-            return HIDDEN;
+            return IInputWidgetTypes.HIDDEN;
         }
     }
 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessageSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessageSection.java
index 07f1698aa..8476673b2 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessageSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessageSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlMessageSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlMessageSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_MESSAGE, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessagesSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessagesSection.java
index 86499fb26..14cdabcaa 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessagesSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlMessagesSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlMessagesSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlMessagesSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_MESSAGES, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputFormatSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputFormatSection.java
index 009893dcf..5011d6062 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputFormatSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputFormatSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlOutputFormatSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlOutputFormatSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_OUTPUTFORMAT, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLabelSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLabelSection.java
index 8216d0521..17fb072e7 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLabelSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLabelSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlOutputLabelSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlOutputLabelSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_OUTPUTLABEL, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLinkSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLinkSection.java
index 358571bf0..338b7add6 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLinkSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputLinkSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlOutputLinkSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlOutputLinkSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_OUTPUTLINK, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputTextSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputTextSection.java
index 2a09d3eba..e46317c5f 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputTextSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlOutputTextSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlOutputTextSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlOutputTextSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_OUTPUTTEXT, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGridSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGridSection.java
index 180ea3365..1c92da34c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGridSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGridSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlPanelGridSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlPanelGridSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_PANELGRID, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGroupSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGroupSection.java
index 57a8bd02e..a9979312c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGroupSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlPanelGroupSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlPanelGroupSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlPanelGroupSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_PANELGROUP, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectBooleanCheckboxSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectBooleanCheckboxSection.java
index f38afaea6..28c358d7b 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectBooleanCheckboxSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectBooleanCheckboxSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectBooleanCheckboxSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectBooleanCheckboxSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTBOOLEANCHECKBOX, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyCheckboxSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyCheckboxSection.java
index df6b50c46..add7c7e6f 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyCheckboxSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyCheckboxSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectManyCheckboxSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectManyCheckboxSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTMANYCHECKBOX, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyListboxSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyListboxSection.java
index d535c1dc0..8ab6d1c94 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyListboxSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyListboxSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectManyListboxSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectManyListboxSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTMANYLISTBOX, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyMenuSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyMenuSection.java
index 7c0dfc2a1..be995238d 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyMenuSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectManyMenuSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectManyMenuSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectManyMenuSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTMANYMENU, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneListboxSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneListboxSection.java
index ccc722b27..47ae3868c 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneListboxSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneListboxSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectOneListboxSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectOneListboxSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTONELISTBOX, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneMenuSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneMenuSection.java
index 349582a02..1d4965ecd 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneMenuSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneMenuSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectOneMenuSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectOneMenuSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTONEMENU, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneRadioSection.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneRadioSection.java
index bd2f5797b..d6b83622a 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneRadioSection.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/JSFHtmlSelectOneRadioSection.java
@@ -20,6 +20,9 @@ import org.eclipse.jst.pagedesigner.properties.attrgroup.AttributeGroupSection;
  */
 public class JSFHtmlSelectOneRadioSection extends AttributeGroupSection
 {
+    /**
+     * The default constructor 
+     */
     public JSFHtmlSelectOneRadioSection()
     {
         super(ITLDConstants.URI_JSF_HTML, IJSFConstants.TAG_SELECTONERADIO, 
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/SectionResources.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/SectionResources.java
index 5e8860d32..c4b2b8233 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/SectionResources.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/sections/SectionResources.java
@@ -30,6 +30,10 @@ public final class SectionResources
         // no external instantiation
     }
 
+    /**
+     * @param key
+     * @return the resource for key or !key! if not found
+     */
     public static String getString(String key)
     {
         try
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/util/JSFUIPluginResourcesUtil.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/util/JSFUIPluginResourcesUtil.java
index bee5bb8a6..82906cec8 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/util/JSFUIPluginResourcesUtil.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/util/JSFUIPluginResourcesUtil.java
@@ -39,6 +39,9 @@ public class JSFUIPluginResourcesUtil extends ResourceUtils
         }
     }
 
+    /**
+     * @return the singleton instance
+     */
     public static JSFUIPluginResourcesUtil getInstance()
     {
         if (_resource == null)