Closed
Bug 137029
Opened 23 years ago
Closed 23 years ago
MathML entries are missing in embedding packages
Categories
(Core :: MathML, defect)
Core
MathML
Tracking
()
RESOLVED
FIXED
mozilla1.0
People
(Reporter: rbs, Assigned: rbs)
Details
(Keywords: topembed+)
Attachments
(1 file, 1 obsolete file)
3.16 KB,
patch
|
adamlock
:
review+
hjtoi-bugzilla
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
http://lxr.mozilla.org/seamonkey/find?string=basebrowser
Also to fix while here: the XHTML DTD entries are missing too.
Updated•23 years ago
|
Keywords: mozilla1.0,
topembed+
All the barsebrowser lists contain res/dtd/*, so all DTDs that the build puts
there should be packaged.
If there are some extra xpt/dll/rdf/etc. packages for MathML you may need to add
those to appropriate places.
> All the barsebrowser lists contain res/dtd/*
Ah OK, I now see. I missed them earlier.
[Other platforms (Mac) have not yet ported the GetBoundingMetrics function:
http://www.mozilla.org/projects/mathml/enable.html]
Comment 4•23 years ago
|
||
what are some MathML URL's we can use to verify that it works in embedding samples?
There are some links to test documents from the MathML project page at
http://www.mozilla.org/projects/mathml/ Dunno if they cover everything, though.
Roger?
Yep, the tests (such as the Torture Test) are sufficient to trigger the
particularities of MathML in the code base. e.g., stretchy characters won't
stretch if |ucvmath| and |res/fons/*| aren't packaged. The style of scripts
(subscript, superscript, accent, fraction, etc) won't reduce if |mathml.css|
isn't packaged.
Note for the m1.0 branch: bug 135895 (a regression in XHTML) causes the MathML
rendering on the m1.0 branch to not match with existing screenshots. (Bug 135895
is a regression w.r.t. <style> in XHTML. It was only fixed on the trunk so far.)
Comment 8•23 years ago
|
||
Roger : Any idea when Bug 135895 is going to be fixed on the Moz 1.0 branch..thanks
No clues. From my cursory reading of it, peterv has been awaiting ADT approval
for some time now.
Assignee | ||
Comment 10•23 years ago
|
||
Seeking r=/sr= for the trunk so that I can check in there and ask for a= on the
branch afterwards.
Summary: MathML entries are missing in embedding packages → [fix, awaiting r/sr/a] MathML entries are missing in embedding packages
Whiteboard: fix, awaiting r/sr/a
Comment on attachment 79387 [details] [diff] [review]
fix for Windows & Unix
This looks good to me, but shouldn't these be added to the Mac packages as
well?
Please check who have given r & sr earlier for packager changes via LXR, and
email them for r/sr.
Assignee | ||
Comment 12•23 years ago
|
||
> This looks good to me
mind giving r/sr to keep the ball rolling then? chak? adamlock? LXR shows your
names.
> but shouldn't these be added to the Mac packages as well?
what do you think in light of comment #3?
Comment 13•23 years ago
|
||
Comment on attachment 79387 [details] [diff] [review]
fix for Windows & Unix
Can you move all the mathml specific stuff into a single block so it's easy to
comment out?
Also if you want to see MathML on that the Mac you'll need to patch the 3 mac
files too.
A further point. The embedding dist is not meant to be everything including the
kitchen sink. It's a *sample* subset of Mozilla. I don't mind if MathML goes in
but its worth bearing in mind for future submissions.
Assignee | ||
Comment 14•23 years ago
|
||
> The embedding dist is not meant to be everything including the kitchen sink.
Oh la la... Check the "View Bug Activity" to see where the keywords came from.
I am doing this patch on request.
Assignee | ||
Comment 15•23 years ago
|
||
Attachment #79387 -
Attachment is obsolete: true
Comment on attachment 80819 [details] [diff] [review]
updated patch
r/sr=heikki
If MathML is not built on Mac/does not work at all, then I am ok without the
mac changes.
Attachment #80819 -
Flags: superreview+
Comment 17•23 years ago
|
||
Comment on attachment 80819 [details] [diff] [review]
updated patch
r=adamlock
Attachment #80819 -
Flags: review+
Assignee | ||
Comment 18•23 years ago
|
||
Checked in the trunk. Have asked drivers for approval on the branch.
Summary: [fix, awaiting r/sr/a] MathML entries are missing in embedding packages → [fix, awaiting a=] MathML entries are missing in embedding packages
Whiteboard: fix, awaiting r/sr/a → fix, checked in the trunk, awaiting a= for the branch
Comment 19•23 years ago
|
||
Attachment #80819 -
Flags: approval+
Assignee | ||
Comment 20•23 years ago
|
||
Now checked in the trunk and the 1.0 branch.
Summary: [fix, awaiting a=] MathML entries are missing in embedding packages → MathML entries are missing in embedding packages
Whiteboard: fix, checked in the trunk, awaiting a= for the branch
You need to log in
before you can comment on or make changes to this bug.
Description
•