Remove { vertical-align: baseline } declarations from mathml.css
authordbarton@mathscribe.com <dbarton@mathscribe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 Aug 2012 04:03:19 +0000 (04:03 +0000)
committerdbarton@mathscribe.com <dbarton@mathscribe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 Aug 2012 04:03:19 +0000 (04:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=95015

Reviewed by Eric Seidel.

Source/WebCore:

The { vertical-align: baseline } declarations in mathml.css had no positive effect, and could
wrongly override a previous { vertical-align: sub } or { vertical-align: super } declaration.

Added 1 test to mathml/presentation/subsup.xhtml.

* css/mathml.css:
(math):
(mrow, mfenced):
(msub, msup):
(msubsup):
(munder, mover, munderover):
    - Remove { vertical-align: baseline } declarations

LayoutTests:

Added 1 test to mathml/presentation/subsup.xhtml.

* mathml/presentation/subsup.xhtml:
* platform/mac/mathml/presentation/subsup-expected.png:
* platform/mac/mathml/presentation/subsup-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/mathml/presentation/subsup.xhtml
LayoutTests/platform/mac/mathml/presentation/subsup-expected.png
LayoutTests/platform/mac/mathml/presentation/subsup-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/css/mathml.css

index f178008..209b60a 100644 (file)
@@ -1,3 +1,16 @@
+2012-08-25  David Barton  <dbarton@mathscribe.com>
+
+        Remove { vertical-align: baseline } declarations from mathml.css
+        https://bugs.webkit.org/show_bug.cgi?id=95015
+
+        Reviewed by Eric Seidel.
+
+        Added 1 test to mathml/presentation/subsup.xhtml.
+
+        * mathml/presentation/subsup.xhtml:
+        * platform/mac/mathml/presentation/subsup-expected.png:
+        * platform/mac/mathml/presentation/subsup-expected.txt:
+
 2012-08-25  Philip Rogers  <pdr@google.com>
 
         Roll out r126056 and r126626
index d3bb657..a2e7560 100644 (file)
 <mo>+</mo><msubsup><mfenced><mfrac><mi>f</mi><mi>g</mi></mfrac></mfenced><mi>n</mi><mi>x</mi></msubsup>
 </math>
 </p>
+
+<p id='t7'>For <a href="https://bugs.webkit.org/show_bug.cgi?id=95015">Bug 95015</a> -
+Remove { vertical-align: baseline } declarations from mathml.css:
+       <math xmlns='http://www.w3.org/1998/Math/MathML'>
+               <msup>
+                       <mi>x</mi>
+                       <msubsup><mi>y</mi><mn>1</mn><mn>2</mn></msubsup>
+               </msup>
+       </math>
+</p>
+
 </body>
 </html>
index e58484e..701b0a7 100644 (file)
Binary files a/LayoutTests/platform/mac/mathml/presentation/subsup-expected.png and b/LayoutTests/platform/mac/mathml/presentation/subsup-expected.png differ
index e4146c8..f4bf7f6 100644 (file)
@@ -1,8 +1,8 @@
 layer at (0,0) size 800x600
   RenderView at (0,0) size 800x600
-layer at (0,0) size 800x262
-  RenderBlock {html} at (0,0) size 800x262
-    RenderBody {body} at (8,16) size 784x230
+layer at (0,0) size 800x298
+  RenderBlock {html} at (0,0) size 800x298
+    RenderBody {body} at (8,16) size 784x266
       RenderBlock {p} at (0,0) size 784x21
         RenderText {#text} at (0,0) size 36x18
           text run at (0,0) width 36: "both: "
@@ -221,6 +221,35 @@ layer at (0,0) size 800x262
                   RenderText {#text} at (0,-7) size 6x18
                     text run at (0,-7) width 6: "n"
         RenderText {#text} at (0,0) size 0x0
+      RenderBlock {p} at (0,246) size 784x20
+        RenderText {#text} at (0,2) size 26x18
+          text run at (0,2) width 26: "For "
+        RenderInline {a} at (0,0) size 71x18 [color=#0000EE]
+          RenderText {#text} at (26,2) size 71x18
+            text run at (26,2) width 71: "Bug 95015"
+        RenderText {#text} at (97,2) size 430x18
+          text run at (97,2) width 430: " - Remove { vertical-align: baseline } declarations from mathml.css: "
+        RenderMathMLMath {math} at (527,0) size 20x17 [padding: 0 1 0 1]
+          RenderMathMLSubSup {msup} at (1,0) size 18x17
+            RenderMathMLBlock (anonymous, inline-block) at (0,8) size 7x9
+              RenderInline {mi} at (0,0) size 7x24
+                RenderText {#text} at (0,-9) size 7x24
+                  text run at (0,-9) width 7: "x"
+            RenderMathMLSubSup {msubsup} at (7,0) size 11x16
+              RenderMathMLBlock (anonymous, inline-block) at (0,0) size 5x13 [padding: 4 0 0 0]
+                RenderInline {mi} at (0,0) size 5x18
+                  RenderText {#text} at (0,-3) size 5x18
+                    text run at (0,-3) width 5: "y"
+              RenderMathMLBlock (anonymous, inline-block) at (6,0) size 5x16
+                RenderMathMLBlock (anonymous, block) at (0,0) size 5x8
+                  RenderInline {mn} at (0,0) size 5x13
+                    RenderText {#text} at (0,-2) size 5x13
+                      text run at (0,-2) width 5: "2"
+                RenderMathMLBlock (anonymous, block) at (0,8) size 5x8
+                  RenderInline {mn} at (0,0) size 5x13
+                    RenderText {#text} at (0,-2) size 5x13
+                      text run at (0,-2) width 5: "1"
+        RenderText {#text} at (0,0) size 0x0
 layer at (238,207) size 5x10 scrollHeight 18
   RenderMathMLBlock (block) {mfenced} at (0,0) size 5x10
 layer at (238,217) size 5x10 scrollHeight 17
index 69b9beb..9a40190 100644 (file)
@@ -1,3 +1,23 @@
+2012-08-25  David Barton  <dbarton@mathscribe.com>
+
+        Remove { vertical-align: baseline } declarations from mathml.css
+        https://bugs.webkit.org/show_bug.cgi?id=95015
+
+        Reviewed by Eric Seidel.
+
+        The { vertical-align: baseline } declarations in mathml.css had no positive effect, and could
+        wrongly override a previous { vertical-align: sub } or { vertical-align: super } declaration.
+
+        Added 1 test to mathml/presentation/subsup.xhtml.
+
+        * css/mathml.css:
+        (math):
+        (mrow, mfenced):
+        (msub, msup):
+        (msubsup):
+        (munder, mover, munderover):
+            - Remove { vertical-align: baseline } declarations
+
 2012-08-25  Roland Takacs  <rtakacs@inf.u-szeged.hu>, Helder Correia  <helder.correia@nokia.com>
 
         [Texmap] Move TextureMapperGL to use GraphicsContext3D
index fa4288f..adb4420 100644 (file)
@@ -14,7 +14,6 @@ math {
     padding: 0px;
     margin: 0px;
     text-align: left;
-    vertical-align: baseline;
     padding-left: 1px;
     padding-right: 1px;
 }
@@ -31,7 +30,6 @@ math[display="block"] {
 mrow, mfenced {
     display: inline-block;
     white-space: nowrap;
-    vertical-align: baseline;
 }
 
 mfenced {
@@ -54,7 +52,6 @@ mfrac {
 
 msub, msup {
     display: inline-block;
-    vertical-align: baseline;
 }
 
 msub > * + * {
@@ -69,7 +66,6 @@ msup > * + * {
 
 msubsup {
     display: inline-block;
-    vertical-align: baseline;
 }
 
 msubsup > * {
@@ -83,7 +79,6 @@ msubsup > * + * {
 
 munder, mover, munderover {
     display: inline-block;
-    vertical-align: baseline;
 }
 
 munderover > * + *, mover > * + *, munder > * + * {