Directional single quotation marks are not rotated in vertical text
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 6 Dec 2014 05:14:12 +0000 (05:14 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 6 Dec 2014 05:14:12 +0000 (05:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=138526

Source/WebCore:

Reviewed by Darin Adler.

In vertical text, directional single quotation marks are not rotated along with
the rest of the letters.

Test: fast/text/vertical-quotation-marks.html

* platform/graphics/FontGlyphs.cpp:
(WebCore::shouldIgnoreRotation):

LayoutTests:

Compare vertical and horizontal renderings.

Reviewed by Darin Adler.

* fast/text/vertical-quotation-marks-expected.html: Added.
* fast/text/vertical-quotation-marks.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@176903 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/text/vertical-quotation-marks-expected.html [new file with mode: 0644]
LayoutTests/fast/text/vertical-quotation-marks.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/FontGlyphs.cpp

index 8404251..8cfe3e8 100644 (file)
@@ -1,3 +1,15 @@
+2014-12-05  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Directional single quotation marks are not rotated in vertical text
+        https://bugs.webkit.org/show_bug.cgi?id=138526
+
+        Compare vertical and horizontal renderings.
+
+        Reviewed by Darin Adler.
+
+        * fast/text/vertical-quotation-marks-expected.html: Added.
+        * fast/text/vertical-quotation-marks.html: Added.
+
 2014-12-05  Dhi Aurrahman  <diorahman@rockybars.com>
 
         Implement parser for :lang pseudo class selector arguments that contain wildcard '*' subtags
diff --git a/LayoutTests/fast/text/vertical-quotation-marks-expected.html b/LayoutTests/fast/text/vertical-quotation-marks-expected.html
new file mode 100644 (file)
index 0000000..84be55d
--- /dev/null
@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+<head>
+</head>
+<body>
+<div style="line-height: 50px; -webkit-transform-origin: left bottom; -webkit-transform: rotate(90deg) translateX(-50px); display: inline-block">
+What&#x2019;s going&#x2018; on?
+</div>
+</body>
+</html
diff --git a/LayoutTests/fast/text/vertical-quotation-marks.html b/LayoutTests/fast/text/vertical-quotation-marks.html
new file mode 100644 (file)
index 0000000..0366350
--- /dev/null
@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+<head>
+</head>
+<body>
+<div style="line-height: 50px; -webkit-writing-mode:vertical-rl;">
+What&#x2019;s going&#x2018; on?
+</div>
+</body>
+</html
index c3a6630..d87a446 100644 (file)
@@ -1,3 +1,18 @@
+2014-12-05  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Directional single quotation marks are not rotated in vertical text
+        https://bugs.webkit.org/show_bug.cgi?id=138526
+
+        Reviewed by Darin Adler.
+
+        In vertical text, directional single quotation marks are not rotated along with
+        the rest of the letters.
+
+        Test: fast/text/vertical-quotation-marks.html
+
+        * platform/graphics/FontGlyphs.cpp:
+        (WebCore::shouldIgnoreRotation):
+
 2014-12-05  Dhi Aurrahman  <diorahman@rockybars.com>
 
         Implement parser for :lang pseudo class selector arguments that contain wildcard '*' subtags
index 3330b23..9321635 100644 (file)
@@ -145,8 +145,7 @@ static bool shouldIgnoreRotation(UChar32 character)
     if (isInRange(character, 0x01100, 0x011FF) || isInRange(character, 0x01401, 0x0167F) || isInRange(character, 0x01800, 0x018FF))
         return true;
 
-    if (character == 0x02016 || character == 0x02018 || character == 0x02019 || character == 0x02020 || character == 0x02021
-        || character == 0x2030 || character == 0x02031)
+    if (character == 0x02016 || character == 0x02020 || character == 0x02021 || character == 0x2030 || character == 0x02031)
         return true;
 
     if (isInRange(character, 0x0203B, 0x0203D) || character == 0x02042 || character == 0x02044 || character == 0x02047