Changeset 7383 in josm for trunk/src/org

Timestamp:

2014-08-13T16:42:17+02:00 (10 years ago)

Author:

bastiK

Message:

applied #10301 - extend display of maxspeed nodes (patch by Klumbumbus)
Include Droid Sans font in the JOSM binary distribution.
This unifies the font rendering on different platforms and allows geometric constructions with text (as demonstrated for maxspeed). Both regular and bold style are available.

Location:

trunk/src/org/openstreetmap/josm

Files:

• gui/MainApplication.java (modified) (2 diffs)
• gui/mappaint/AreaElemStyle.java (modified) (2 diffs)
• gui/mappaint/BoxTextElemStyle.java (modified) (1 diff)
• gui/mappaint/ElemStyle.java (modified) (3 diffs)
• gui/mappaint/ElemStyles.java (modified) (2 diffs)
• gui/mappaint/LineTextElemStyle.java (modified) (2 diffs)
• gui/mappaint/TextElement.java (modified) (1 diff)
• tools/FontsManager.java (added)

trunk/src/org/openstreetmap/josm/gui/MainApplication.java

@@ -60 +60 @@
 import org.openstreetmap.josm.plugins.PluginInformation;
 import org.openstreetmap.josm.tools.BugReportExceptionHandler;
+import org.openstreetmap.josm.tools.FontsManager;
 import org.openstreetmap.josm.tools.I18n;
 import org.openstreetmap.josm.tools.ImageProvider;
@@ -346 +347 @@
         Main.pref.updateSystemProperties();
 
+        FontsManager.initialize();
+        
         final JFrame mainFrame = new JFrame(tr("Java OpenStreetMap Editor"));
         Main.parent = mainFrame;

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

@@ -34 +34 @@
     }
 
-    public static AreaElemStyle create(Cascade c) {
+    public static AreaElemStyle create(final Environment env) {
+        final Cascade c = env.mc.getCascade(env.layer);
         MapImage fillImage = null;
         Color color = null;
@@ -75 +76 @@
         Keyword textPos = c.get(TEXT_POSITION, null, Keyword.class);
         if (textPos == null || "center".equals(textPos.val)) {
-            text = TextElement.create(c, PaintColors.AREA_TEXT.get(), true);
+            text = TextElement.create(env, PaintColors.AREA_TEXT.get(), true);
         }
 

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

@@ -111 +111 @@
         Cascade c = env.mc.getCascade(env.layer);
 
-        TextElement text = TextElement.create(c, DEFAULT_TEXT_COLOR, false);
+        TextElement text = TextElement.create(env, DEFAULT_TEXT_COLOR, false);
         if (text == null) return null;
         // Skip any primitives that don't have text to draw. (Styles are recreated for any tag change.)

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

@@ -112 +112 @@
                 n = DEFAULT_FONT_NAME;
                 if (n == null) {
-                    DEFAULT_FONT_NAME = n = Main.pref.get("mappaint.font", "Helvetica");
+                    DEFAULT_FONT_NAME = n = Main.pref.get("mappaint.font", "Droid Sans");
                 }
             }
@@ -174 +174 @@
     }
 
-    protected static Font getFont(Cascade c) {
+    protected static Font getFont(Cascade c, String s) {
         String name = c.get("font-family", getDefaultFontName(), String.class);
         float size = c.get("font-size", getDefaultFontSize(), Float.class);
@@ -185 +185 @@
             style = Font.ITALIC;
         }
-        return getCachedFont(name, style | weight, Math.round(size));
+        Font f = getCachedFont(name, style | weight, Math.round(size));
+        if (f.canDisplayUpTo(s) == -1)
+            return f;
+        else {
+            // fallback if the string contains characters that cannot be
+            // rendered by the selected font
+            return getCachedFont("SansSerif", style | weight, Math.round(size));
+        }
     }
 

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

@@ -337 +337 @@
             Cascade c = e.getValue();
             if (osm instanceof Way) {
-                addIfNotNull(sl, AreaElemStyle.create(c));
+                addIfNotNull(sl, AreaElemStyle.create(env));
                 addIfNotNull(sl, RepeatImageElemStyle.create(env));
                 addIfNotNull(sl, LineElemStyle.createLine(env));
@@ -354 +354 @@
             } else if (osm instanceof Relation) {
                 if (((Relation)osm).isMultipolygon()) {
-                    addIfNotNull(sl, AreaElemStyle.create(c));
+                    addIfNotNull(sl, AreaElemStyle.create(env));
                     addIfNotNull(sl, RepeatImageElemStyle.create(env));
                     addIfNotNull(sl, LineElemStyle.createLine(env));

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

@@ -18 +18 @@
         this.text = text;
     }
-    public static LineTextElemStyle create(Environment env) {
-        Cascade c = env.mc.getCascade(env.layer);
+    public static LineTextElemStyle create(final Environment env) {
+        final Cascade c = env.mc.getCascade(env.layer);
 
         Keyword textPos = c.get(TEXT_POSITION, null, Keyword.class);
@@ -25 +25 @@
             return null;
 
-        TextElement text = TextElement.create(c, PaintColors.TEXT.get(), false);
+        TextElement text = TextElement.create(env, PaintColors.TEXT.get(), false);
         if (text == null)
             return null;

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

@@ -116 +116 @@
      * @throws IllegalArgumentException thrown if {@code defaultTextColor} is null
      */
-    public static TextElement create(Cascade c, Color defaultTextColor, boolean defaultAnnotate)  throws IllegalArgumentException{
+    public static TextElement create(Environment env, Color defaultTextColor, boolean defaultAnnotate)  throws IllegalArgumentException{
         CheckParameterUtil.ensureParameterNotNull(defaultTextColor);
+        Cascade c = env.mc.getCascade(env.layer);
 
         LabelCompositionStrategy strategy = buildLabelCompositionStrategy(c, defaultAnnotate);
         if (strategy == null) return null;
-        Font font = ElemStyle.getFont(c);
+        String s = strategy.compose(env.osm);
+        if (s == null) return null;
+        Font font = ElemStyle.getFont(c, s);
 
         float xOffset = 0;