Do not rotate non-CJK characters in vertical writing mode (fixes #3506)

mapbox · Nov 4, 2016 · eedb1a0 · eedb1a0
1 parent 97ef972
commit eedb1a0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/js/symbol/shaping.js b/js/symbol/shaping.js
@@ -54,7 +54,7 @@ function shapeText(text, glyphs, maxWidth, lineHeight, horizontalAlign, vertical
 
         if (!glyph && codePoint !== newLine) continue;
 
-        if (writingMode === WritingMode.horizontal) {
+        if (!scriptDetection.charAllowsVerticalWritingMode(text[i]) || writingMode === WritingMode.horizontal) {
             positionedGlyphs.push(new PositionedGlyph(codePoint, x, yOffset, glyph, 0));
             if (glyph) x += glyph.advance + spacing;
 

diff --git a/js/util/script_detection.js b/js/util/script_detection.js
@@ -12,7 +12,7 @@ module.exports.allowsIdeographicBreaking = function(chars) {
 
 module.exports.allowsVerticalWritingMode = function(chars) {
     for (const char of chars) {
-        if (charAllowsVerticalWritingMode(char.charCodeAt(0))) return true;
+        if (exports.charAllowsVerticalWritingMode(char.charCodeAt(0))) return true;
     }
     return false;
 };
@@ -45,7 +45,7 @@ module.exports.charAllowsIdeographicBreaking = function(char) {
     return false;
 };
 
-function charAllowsVerticalWritingMode(char) {
+exports.charAllowsVerticalWritingMode = function(char) {
     // Return early for characters outside all ranges that allow the vertical
     // writing mode.
     if (char < 0x1100) return false;