Changeset 3992 in josm

Timestamp:

Mar 14, 2011 12:27:46 AM (2 years ago)

Author:

bastiK

Message:

applied #6107 - MapCSS enhancement (patch by anonymous)

Location:

trunk/src/org/openstreetmap/josm/gui/mappaint

Files:

• AreaElemStyle.java (modified) (1 diff)
• LineElemStyle.java (modified) (1 diff)
• NodeElemStyle.java (modified) (1 diff)
• TextElement.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/AreaElemStyle.java

@@ -80 +80 @@
         Keyword textPos = c.get("text-position", null, Keyword.class);
         if (textPos == null || Utils.equal(textPos.val, "center")) {
-            text = TextElement.create(c, PaintColors.AREA_TEXT.get());
+            text = TextElement.create(c, PaintColors.AREA_TEXT.get(), true);
         }
         

trunk/src/org/openstreetmap/josm/gui/mappaint/LineElemStyle.java

@@ -185 +185 @@
             Keyword textPos = c.get("text-position", null, Keyword.class);
             if (textPos == null || equal(textPos.val, "line")) {
-                text = TextElement.create(c, PaintColors.TEXT.get());
+                text = TextElement.create(c, PaintColors.TEXT.get(), false);
             }
         }

trunk/src/org/openstreetmap/josm/gui/mappaint/NodeElemStyle.java

@@ -177 +177 @@
         }
 
-        if (icon == null && symbol == null && !allowOnlyText)
-            return null;
-
         NodeTextElement text = null;
-        TextElement te = TextElement.create(c, DEFAULT_TEXT_COLOR);
+        TextElement te = TextElement.create(c, DEFAULT_TEXT_COLOR, symbol == null && icon == null);
         // optimization: if we neither have a symbol, nor an icon, nor a text element
         // we don't have to check for the remaining style properties and we don't

trunk/src/org/openstreetmap/josm/gui/mappaint/TextElement.java

@@ -80 +80 @@
      * @return the label composition strategy
      */
-    protected static LabelCompositionStrategy buildLabelCompositionStrategy(Cascade c){
+    protected static LabelCompositionStrategy buildLabelCompositionStrategy(Cascade c, boolean defaultAnnotate){
         Keyword textKW = c.get("text", null, Keyword.class, true);
         if (textKW == null) {
             String textKey = c.get("text", null, String.class);
-            if (textKey == null) return null;
+            if (textKey == null) 
+                return defaultAnnotate ? AUTO_LABEL_COMPOSITION_STRATEGY : null;
             return new TagLookupCompositionStrategy(textKey);
         } else if (textKW.val.equals("auto"))
@@ -102 +103 @@
      * @throws IllegalArgumentException thrown if {@code defaultTextColor} is null
      */
-    public static TextElement create(Cascade c, Color defaultTextColor)  throws IllegalArgumentException{
+    public static TextElement create(Cascade c, Color defaultTextColor, boolean defaultAnnotate)  throws IllegalArgumentException{
         CheckParameterUtil.ensureParameterNotNull(defaultTextColor);
 
-        LabelCompositionStrategy strategy = buildLabelCompositionStrategy(c);
+        LabelCompositionStrategy strategy = buildLabelCompositionStrategy(c, defaultAnnotate);
         Font font = ElemStyle.getFont(c);