Index: trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetType.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetType.java	(revision 8482)
+++ trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetType.java	(revision 8484)
@@ -10,7 +10,11 @@
  */
 public enum TaggingPresetType {
+    /** Node */
     NODE(/* ICON */ "Mf_node", "node"),
+    /** Way */
     WAY(/* ICON */ "Mf_way", "way"),
+    /** Relation */
     RELATION(/* ICON */ "Mf_relation", "relation"),
+    /** Closed way */
     CLOSEDWAY(/* ICON */ "Mf_closedway", "closedway");
     private final String iconName;
@@ -18,20 +22,38 @@
 
     TaggingPresetType(String iconName, String name) {
-        this.iconName = iconName;
+        this.iconName = iconName + ".svg";
         this.name = name;
     }
 
+    /**
+     * Replies the SVG icon name.
+     * @return the SVG icon name
+     */
     public String getIconName() {
         return iconName;
     }
 
+    /**
+     * Replies the name, as used in XML presets.
+     * @return the name: "node", "way", "relation" or "closedway"
+     */
     public String getName() {
         return name;
     }
 
+    /**
+     * Determines the {@code TaggingPresetType} of a given primitive.
+     * @param p The OSM primitive
+     * @return the {@code TaggingPresetType} of {@code p}
+     */
     public static TaggingPresetType forPrimitive(OsmPrimitive p) {
         return forPrimitiveType(p.getDisplayType());
     }
 
+    /**
+     * Determines the {@code TaggingPresetType} of a given primitive type.
+     * @param type The OSM primitive type
+     * @return the {@code TaggingPresetType} of {@code type}
+     */
     public static TaggingPresetType forPrimitiveType(OsmPrimitiveType type) {
         if (type == OsmPrimitiveType.NODE) return NODE;
@@ -43,4 +65,9 @@
     }
 
+    /**
+     * Determines the {@code TaggingPresetType} from a given string.
+     * @param type The OSM primitive type as string ("node", "way", "relation" or "closedway")
+     * @return the {@code TaggingPresetType} from {@code type}
+     */
     public static TaggingPresetType fromString(String type) {
         for (TaggingPresetType t : TaggingPresetType.values()) {
@@ -51,4 +78,3 @@
         return null;
     }
-
 }