Bad spacing inside MathML formulas when text-indent is specified
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 16:26:11 +0000 (16:26 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 16:26:11 +0000 (16:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=106600

Patch by Frédéric Wang <fred.wang@free.fr> on 2013-05-17
Reviewed by Martin Robinson.

Source/WebCore:

When MathML is used in a HTML page that modifies the CSS text-indent,
large gaps appear inside the mathematical expressions. Resetting it to
0 on the math root (as Gecko does) fixes the issue.

Tests: mathml/presentation/text-indent.html
       mathml/presentation/text-indent-expected.html

* css/mathml.css:
(math): reset text-indent to its default value.

LayoutTests:

New test to verify that the text-indent property is reset to 0 on
the MathML root.

* mathml/presentation/text-indent-expected.html: Added.
* mathml/presentation/text-indent.html: Added.

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

LayoutTests/ChangeLog
LayoutTests/mathml/presentation/text-indent-expected.html [new file with mode: 0644]
LayoutTests/mathml/presentation/text-indent.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/css/mathml.css

index 38b4e53..8f6fbe4 100644 (file)
@@ -1,3 +1,16 @@
+2013-05-17  Frédéric Wang  <fred.wang@free.fr>
+
+        Bad spacing inside MathML formulas when text-indent is specified
+        https://bugs.webkit.org/show_bug.cgi?id=106600
+
+        Reviewed by Martin Robinson.
+
+        New test to verify that the text-indent property is reset to 0 on
+        the MathML root.
+
+        * mathml/presentation/text-indent-expected.html: Added.
+        * mathml/presentation/text-indent.html: Added.
+
 2013-05-16  Jer Noble  <jer.noble@apple.com>
 
         Some media/track tests fail or assert on Mac
diff --git a/LayoutTests/mathml/presentation/text-indent-expected.html b/LayoutTests/mathml/presentation/text-indent-expected.html
new file mode 100644 (file)
index 0000000..5c0959c
--- /dev/null
@@ -0,0 +1,15 @@
+<!DOCTYPE html>
+<html>
+<body style="text-indent: 2em">
+
+  <math style="text-indent: 0">
+    <mn>2</mn>
+    <mo>+</mo>
+    <mfrac>
+      <mi>x</mi>
+      <mi>y</mi>
+    </mfrac>
+  </math>
+
+</body>
+</html>
diff --git a/LayoutTests/mathml/presentation/text-indent.html b/LayoutTests/mathml/presentation/text-indent.html
new file mode 100644 (file)
index 0000000..6d1497b
--- /dev/null
@@ -0,0 +1,15 @@
+<!DOCTYPE html>
+<html>
+<body style="text-indent: 2em">
+
+  <math>
+    <mn>2</mn>
+    <mo>+</mo>
+    <mfrac>
+      <mi>x</mi>
+      <mi>y</mi>
+    </mfrac>
+  </math>
+
+</body>
+</html>
index b4ba82b..7b4932b 100644 (file)
@@ -1,3 +1,20 @@
+2013-05-17  Frédéric Wang  <fred.wang@free.fr>
+
+        Bad spacing inside MathML formulas when text-indent is specified
+        https://bugs.webkit.org/show_bug.cgi?id=106600
+
+        Reviewed by Martin Robinson.
+
+        When MathML is used in a HTML page that modifies the CSS text-indent,
+        large gaps appear inside the mathematical expressions. Resetting it to
+        0 on the math root (as Gecko does) fixes the issue.
+
+        Tests: mathml/presentation/text-indent.html
+               mathml/presentation/text-indent-expected.html
+
+        * css/mathml.css:
+        (math): reset text-indent to its default value.
+
 2013-05-17  Alberto Garcia  <agarcia@igalia.com>
 
         Fix code that expects Page::chrome() to return a pointer
index 98bfe93..7ce45b7 100644 (file)
@@ -3,6 +3,7 @@
 math {
     -webkit-line-box-contain: glyphs replaced;
     line-height: 0;
+    text-indent: 0;
 }
 mtext {
     line-height: 1.0;