add twitter-cldr for bidirectional text support

mapbox · Dec 15, 2015 · ed975d5 · ed975d5
1 parent ba9d109
commit ed975d5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/js/symbol/resolve_text.js b/js/symbol/resolve_text.js
@@ -1,6 +1,8 @@
 'use strict';
 
 var resolveTokens = require('../util/token');
+var TwitterCLDR = require('../../node_modules/twitter_cldr/min/en.min.js');
+var bidi = TwitterCLDR.Bidi;
 
 module.exports = resolveText;
 
@@ -29,6 +31,8 @@ function resolveText(features, layoutProperties, codepoints) {
             text = text.toLocaleLowerCase();
         }
 
+        text = bidi.from_string(text).reorder_visually().toString();
+
         for (var j = 0; j < text.length; j++) {
             codepoints[text.charCodeAt(j)] = true;
         }

diff --git a/package.json b/package.json
@@ -24,6 +24,7 @@
     "rbush": "^1.4.0",
     "request": "^2.39.0",
     "resolve-url": "^0.2.1",
+    "twitter_cldr": "^1.5.0",
     "unassertify": "^2.0.0",
     "unitbezier": "^0.0.0",
     "vector-tile": "^1.2.0",