Changeset 16253 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint

Timestamp:

2020-04-11T09:55:52+02:00 (4 years ago)

Author:

simon04

Message:

fix #18468 - MapCSS: add support for text-rotation

Location:

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

Files:

• StyleKeys.java (modified) (1 diff)
• styleelement/NodeElement.java (modified) (1 diff)
• styleelement/TextLabel.java (modified) (9 diffs)

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

@@ -92 +92 @@
     String ICON_ROTATION = "icon-rotation";
     /**
+     * MapCSS text-rotation property key
+     */
+    String TEXT_ROTATION = "text-rotation";
+    /**
      * MapCSS icon-width property key
      */

trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java

@@ -88 +88 @@
      */
     public static RotationAngle createRotationAngle(Environment env) {
+        return createRotationAngle(env, ICON_ROTATION);
+    }
+
+    /**
+     * Reads the text-rotation property and creates a rotation angle from it.
+     * @param env The environment
+     * @return The angle
+     * @since 16253
+     */
+    public static RotationAngle createTextRotationAngle(Environment env) {
+        return createRotationAngle(env, TEXT_ROTATION);
+    }
+
+    private static RotationAngle createRotationAngle(Environment env, String key) {
         Cascade c = env.mc.getCascade(env.layer);
 
         RotationAngle rotationAngle = RotationAngle.NO_ROTATION;
-        final Float angle = c.get(ICON_ROTATION, null, Float.class, true);
+        final Float angle = c.get(key, null, Float.class, true);
         if (angle != null) {
             rotationAngle = RotationAngle.buildStaticRotation(angle);
         } else {
-            final Keyword rotationKW = c.get(ICON_ROTATION, null, Keyword.class);
+            final Keyword rotationKW = c.get(key, null, Keyword.class);
             if (rotationKW != null) {
                 if ("way".equals(rotationKW.val)) {

trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/TextLabel.java

@@ -19 +19 @@
 import org.openstreetmap.josm.tools.CheckParameterUtil;
 import org.openstreetmap.josm.tools.ColorHelper;
+import org.openstreetmap.josm.tools.RotationAngle;
 
 /**
@@ -40 +41 @@
     public Font font;
     /**
+     * The rotation angle to be used when rendering
+     */
+    public RotationAngle rotationAngle;
+    /**
      * The color to draw the text in, includes alpha.
      */
@@ -58 +63 @@
      * If null, no label is rendered.
      * @param font the font to be used. Must not be null.
+     * @param rotationAngle the rotation angle to be used. Must not be null.
      * @param color the color to be used. Must not be null
      * @param haloRadius halo radius
      * @param haloColor halo color
      */
-    protected TextLabel(LabelCompositionStrategy strategy, Font font, Color color, Float haloRadius,
-            Color haloColor) {
+    protected TextLabel(LabelCompositionStrategy strategy, Font font, RotationAngle rotationAngle,
+                        Color color, Float haloRadius, Color haloColor) {
         this.labelCompositionStrategy = strategy;
         this.font = Objects.requireNonNull(font, "font");
+        this.rotationAngle = Objects.requireNonNull(rotationAngle, "rotationAngle");
         this.color = Objects.requireNonNull(color, "color");
         this.haloRadius = haloRadius;
@@ -79 +86 @@
         this.labelCompositionStrategy = other.labelCompositionStrategy;
         this.font = other.font;
+        this.rotationAngle = other.rotationAngle;
         this.color = other.color;
         this.haloColor = other.haloColor;
@@ -137 +145 @@
         if (s == null) return null;
         Font font = StyleElement.getFont(c, s);
+        RotationAngle rotationAngle = NodeElement.createTextRotationAngle(env);
 
         Color color = c.get(TEXT_COLOR, defaultTextColor, Color.class);
@@ -153 +162 @@
         }
 
-        return new TextLabel(strategy, font, color, haloRadius, haloColor);
+        return new TextLabel(strategy, font, rotationAngle, color, haloRadius, haloColor);
     }
 
@@ -199 +208 @@
         sb.append("labelCompositionStrategy=").append(labelCompositionStrategy)
           .append(" font=").append(font)
+          .append(" rotationAngle=").append(rotationAngle)
           .append(" color=").append(ColorHelper.color2html(color));
         if (haloRadius != null) {
@@ -209 +219 @@
     @Override
     public int hashCode() {
-        return Objects.hash(labelCompositionStrategy, font, color, haloRadius, haloColor);
+        return Objects.hash(labelCompositionStrategy, font, rotationAngle, color, haloRadius, haloColor);
     }
 
@@ -219 +229 @@
         return Objects.equals(labelCompositionStrategy, textLabel.labelCompositionStrategy) &&
                 Objects.equals(font, textLabel.font) &&
+                Objects.equals(rotationAngle, textLabel.rotationAngle) &&
                 Objects.equals(color, textLabel.color) &&
                 Objects.equals(haloRadius, textLabel.haloRadius) &&