Make sure font family works for SLD as well

MapStory · Mar 3, 2015 · d3ffa55 · d3ffa55
1 parent dd9b49c
commit d3ffa55
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/edit/style/SLDStyleConverter.js b/lib/edit/style/SLDStyleConverter.js
@@ -214,6 +214,16 @@ exports.SLDStyleConverter = function() {
             };
         },
         createTextSymbolizer: function(style) {
+            var fontFamily;
+            if (style.label.fontFamily === 'serif') {
+                fontFamily  = 'Serif';
+            } else if (style.label.fontFamily === 'sans-serif') {
+                fontFamily = 'SansSerif';
+            } else if (style.label.fontFamily === 'cursive') {
+                fontFamily = 'Comic Sans MS';
+            } else if (style.label.fontFamily === 'monospace') {
+                fontFamily = 'Courier New';
+            }
             return {
                 name: {
                     localPart: 'TextSymbolizer',
@@ -243,7 +253,7 @@ exports.SLDStyleConverter = function() {
                     font: {
                         cssParameter: [{
                                 name: 'font-family',
-                                content: [style.label.fontFamily]
+                                content: [fontFamily]
                             }, {
                                 name: 'font-size',
                                 content: [String(style.label.fontSize)]

diff --git a/test/test-SLDStyleConverter.js b/test/test-SLDStyleConverter.js
@@ -105,7 +105,7 @@ describe('SLDStyleConverter', function() {
             "label": {
                 "attribute": "foo",
                 "fillColor": "#000000",
-                "fontFamily": "Serif",
+                "fontFamily": "serif",
                 "fontSize": 10,
                 "fontStyle": "normal",
                 "fontWeight": "normal"