Bug 95015 - Remove { vertical-align: baseline } declarations from mathml.css
Summary: Remove { vertical-align: baseline } declarations from mathml.css
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: MathML (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dave Barton
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-25 16:23 PDT by Dave Barton
Modified: 2012-08-25 21:03 PDT (History)
11 users (show)

See Also:

Attachments
The <msubsup> expression should appear as a superscript. (114 bytes, text/html)
2012-08-25 16:23 PDT, Dave Barton 		no flags Details
Patch (47.50 KB, patch)
2012-08-25 17:08 PDT, Dave Barton 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Barton 2012-08-25 16:23:24 PDT 
Created attachment 160577 [details]
The <msubsup> expression should appear as a superscript.

The { vertical-align: baseline } declarations in mathml.css have no positive affect, and can wrongly override a previous { vertical-align: sub } or { vertical-align: super } declaration.
Comment 1 Dave Barton 2012-08-25 17:08:48 PDT 
Created attachment 160581 [details]
Patch
Comment 2 Dave Barton 2012-08-25 19:18:05 PDT 
This is the first in a series of small patches to clean up mathml.css. Thanks for any review!
Comment 3 Eric Seidel (no email) 2012-08-25 20:52:51 PDT 
Comment on attachment 160581 [details]
Patch

LGTM.
Comment 4 WebKit Review Bot 2012-08-25 21:02:58 PDT 
Comment on attachment 160581 [details]
Patch

Clearing flags on attachment: 160581

Committed r126698: <http://trac.webkit.org/changeset/126698>
Comment 5 WebKit Review Bot 2012-08-25 21:03:02 PDT 
All reviewed patches have been landed.  Closing bug.