Bug 95404 - mathml.css: Add more { white-space: nowrap } declarations
Summary: mathml.css: Add more { white-space: nowrap } declarations
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: 96009
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-29 16:56 PDT by Dave Barton
Modified: 2012-09-06 12:51 PDT (History)
10 users (show)

See Also:


Attachments
"y" should appear as a superscript in x^y (131 bytes, text/html)
2012-08-29 16:56 PDT, Dave Barton
no flags Details
Patch (90.55 KB, patch)
2012-08-29 17:51 PDT, Dave Barton
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Barton 2012-08-29 16:56:00 PDT
Created attachment 161361 [details]
"y" should appear as a superscript in x^y

mathml.css declares { white-space: nowrap } for mrow, mfenced, msqrt. It is also needed for other tags such as msub, msup, msubsup.
Comment 1 Dave Barton 2012-08-29 17:51:36 PDT
Created attachment 161371 [details]
Patch
Comment 2 Dave Barton 2012-09-06 11:25:41 PDT
Any comments or review + or - would be groovy. :)

This short bug+patch basically prevents line breaks between e.g. a base and its subscript and/or superscript.

I did a --cc=eric@webkit.org,jchaffraix@webkit.org,mitz@webkit.org,christophe.dumez@intel.com on my webkit-patch upload of the patch a week ago, but apparently it didn't take effect. Perhaps that option only works when creating a new bug report?
Comment 3 Eric Seidel (no email) 2012-09-06 11:51:02 PDT
(In reply to comment #2)
> Any comments or review + or - would be groovy. :)
> 
> This short bug+patch basically prevents line breaks between e.g. a base and its subscript and/or superscript.
> 
> I did a --cc=eric@webkit.org,jchaffraix@webkit.org,mitz@webkit.org,christophe.dumez@intel.com on my webkit-patch upload of the patch a week ago, but apparently it didn't take effect. Perhaps that option only works when creating a new bug report?

Yes, sadly.  https://bugs.webkit.org/show_bug.cgi?id=38823
Comment 4 Eric Seidel (no email) 2012-09-06 11:51:25 PDT
Comment on attachment 161371 [details]
Patch

Works for me.
Comment 5 WebKit Review Bot 2012-09-06 12:04:34 PDT
Comment on attachment 161371 [details]
Patch

Clearing flags on attachment: 161371

Committed r127769: <http://trac.webkit.org/changeset/127769>
Comment 6 WebKit Review Bot 2012-09-06 12:04:39 PDT
All reviewed patches have been landed.  Closing bug.