Closed
Bug 130282
Opened 23 years ago
Closed 22 years ago
Torture Test #13 improvements
Categories
(Core :: MathML, defect)
Core
MathML
Tracking
()
RESOLVED
FIXED
mozilla1.1beta
People
(Reporter: xyzzy, Assigned: rbs)
References
()
Details
Attachments
(4 files)
(deleted),
application/xhtml+xml
|
Details | |
(deleted),
patch
|
roc
:
review+
waterson
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details | |
(deleted),
image/gif
|
Details |
Okay, you wanted nits, you got 'em:
The left hand side of this example (the image) renders the radicals so as to
keep the vertical spacing between bars uniform. The MathML version is not
uniform in this vertical spacing. Also, the right-side ends of these bars do
not line up. I noticed this, but needed to magnify to confirm.
TeX seems to try to keep the vertical spacing uniform, while Mozilla is trying
to keep the bottom of the radical glyphs uniform. See, e.g., attachment 90346 [details] from
bug 121493. Which way do you prefer?
Cc:ing roc+moz/waterson for r/sr
Status: NEW → ASSIGNED
OS: Windows 98 → All
Hardware: PC → All
Target Milestone: --- → mozilla1.1beta
The patch only concentrate on the vertical stacking. Getting neat horizontal
widths for the rule of the radical also depend on how well frames align on
pixel boundaries -- it is too complex/unreliable to have a fix for that part,
and so will remain as such.
Comment on attachment 90910 [details] [diff] [review]
patch to beautify the rendering
r=roc+moz
Attachment #90910 -
Flags: review+
Comment 8•22 years ago
|
||
Comment on attachment 90910 [details] [diff] [review]
patch to beautify the rendering
sr=waterson
Attachment #90910 -
Flags: superreview+
Comment 9•22 years ago
|
||
Comment on attachment 90910 [details] [diff] [review]
patch to beautify the rendering
a=asa (on behalf of drivers) for checkin to the 1.1 trunk.
Attachment #90910 -
Flags: approval+
Assignee | ||
Comment 10•22 years ago
|
||
checked on the trunk
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•