2009-11-20 Roland Steiner <rolandsteiner@chromium.org>
authorrolandsteiner@chromium.org <rolandsteiner@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Nov 2009 05:44:06 +0000 (05:44 +0000)
committerrolandsteiner@chromium.org <rolandsteiner@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Nov 2009 05:44:06 +0000 (05:44 +0000)
WebCore: Change default stylesheet to reset text indentation for ruby elements by default.
(https://bugs.webkit.org/show_bug.cgi?id=31247)

Reviewed by Darin Adler.

Test: fast/ruby/ruby-text-indent.html

* css/html.css: reset text-indent on <ruby> and <rt> elements

LayoutTests: Test checking that CSS 'text-indent' does not affect ruby rendering by default.
(https://bugs.webkit.org/show_bug.cgi?id=31247)

Reviewed by Darin Adler.

* fast/ruby/ruby-text-indent-expected.txt: Added.
* fast/ruby/ruby-text-indent.html: Added.

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

LayoutTests/ChangeLog
LayoutTests/fast/ruby/ruby-text-indent-expected.txt [new file with mode: 0644]
LayoutTests/fast/ruby/ruby-text-indent.html [new file with mode: 0644]
WebCore/ChangeLog
WebCore/css/html.css

index e4b9a56..850bfa9 100644 (file)
@@ -1,3 +1,13 @@
+2009-11-20  Roland Steiner  <rolandsteiner@chromium.org>
+
+        Reviewed by Darin Adler.
+
+        Test checking that CSS 'text-indent' does not affect ruby rendering by default.
+        (https://bugs.webkit.org/show_bug.cgi?id=31247)
+
+        * fast/ruby/ruby-text-indent-expected.txt: Added.
+        * fast/ruby/ruby-text-indent.html: Added.
+
 2009-11-19  Beth Dakin  <bdakin@apple.com>
 
         Reviewed by Darin Adler.
diff --git a/LayoutTests/fast/ruby/ruby-text-indent-expected.txt b/LayoutTests/fast/ruby/ruby-text-indent-expected.txt
new file mode 100644 (file)
index 0000000..2b4d325
--- /dev/null
@@ -0,0 +1,11 @@
+This tests that ruby base and text are not affected by CSS text-indent.
+
+This is indented by 50 pixel
+"text-indent: 50px;"
+, but the ruby should flow without extra space.
+
+text-indent for <p>: 50px
+
+text-indent for <ruby>: 0px
+
+text-indent for <rt>: 0px
diff --git a/LayoutTests/fast/ruby/ruby-text-indent.html b/LayoutTests/fast/ruby/ruby-text-indent.html
new file mode 100644 (file)
index 0000000..01e0ac6
--- /dev/null
@@ -0,0 +1,28 @@
+<head>
+<script>
+function test()
+{
+    if (window.layoutTestController)
+        layoutTestController.dumpAsText();
+    var testp = document.getElementById("p");
+    var testr = document.getElementById("r");
+    var testt = document.getElementById("t");
+    var indentp = window.getComputedStyle(testp, null).getPropertyValue("text-indent");
+    var indentr = window.getComputedStyle(testr, null).getPropertyValue("text-indent");
+    var indentt = window.getComputedStyle(testt, null).getPropertyValue("text-indent");
+    var resultp = document.getElementById("resultp");
+    var resultr = document.getElementById("resultr");
+    var resultt = document.getElementById("resultt");
+    resultp.innerHTML = indentp;
+    resultr.innerHTML = indentr;
+    resultt.innerHTML = indentt;
+}
+</script>
+</head>
+<body onload="test()">
+<p>This tests that ruby base and text are not affected by CSS text-indent.</p>
+<p style="text-indent: 50px" id="p">This is <ruby id="r">indented by 50 pixel<rt id="t">&quot;text-indent: 50px;&quot;</rt></ruby>, but the ruby should flow without extra space.</p>
+<p>text-indent for &lt;p&gt;: <span id="resultp">FAILED!</span></p>
+<p>text-indent for &lt;ruby&gt;: <span id="resultr">FAILED!</span></p>
+<p>text-indent for &lt;rt&gt;: <span id="resultt">FAILED!</span></p>
+</body>
index 6e80b2a..cde6cd5 100644 (file)
@@ -1,3 +1,14 @@
+2009-11-20  Roland Steiner  <rolandsteiner@chromium.org>
+
+        Reviewed by Darin Adler.
+
+        Change default stylesheet to reset text indentation for ruby elements by default.
+        (https://bugs.webkit.org/show_bug.cgi?id=31247)
+
+        Test: fast/ruby/ruby-text-indent.html
+
+        * css/html.css: reset text-indent on <ruby> and <rt> elements 
+
 2009-11-19  Brian Weinstein  <bweinstein@apple.com>
 
         Reviewed by Darin Adler.
index c89a000..1132d24 100644 (file)
@@ -609,9 +609,13 @@ a:-webkit-any-link:active {
 
 /* HTML5 ruby elements */
 
+ruby, rt {
+    text-indent: 0; /* blocks used for ruby rendering should not trigger this */ 
+}
+
 ruby > rt {
     display: block;
-    font-size: 60%;         /* make slightly larger than 50% for better readability */
+    font-size: 60%; /* make slightly larger than 50% for better readability */
     text-align: center;
 }