Closed Bug 708072 (test-stretchy) Opened 9 years ago Closed 9 years ago

Test page for stretchy and large operators

Categories

(Core :: MathML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: fredw, Assigned: fredw)

References

Details

Attachments

(2 files)

Attached patch PatchSplinter Review
I open this bug to create a test generated from our operator dictionary, for verifying stretchy and large operators.

Unfortunately, it seems that the dynamically generated content is not well rendered, so I 'll give a static version of the test page.
Assignee: nobody → fred.wang
Comment on attachment 579433 [details] [diff] [review]
Patch

I ask a review now, since fixing the issue with dynamically generated content seems out of the scope of this bug.
Attachment #579433 - Flags: review?(karlt)
Comment on attachment 579433 [details] [diff] [review]
Patch

Looks good thanks.

Some things to consider as possible future improvements:

updateOperatorDictionary.pl would be easier to follow if the array index
constants were names, or, perhaps simpler, the data could be collected into
hash tables with the keys being 'description', 'lspace', etc.

I wonder whether it would be worthwhile considering operators that are
stretchy (have a direction) but are not stretchy by default.  e.g. &Sum.
Attachment #579433 - Flags: review?(karlt) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9dfb1c2c5586
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 755541
You need to log in before you can comment on or make changes to this bug.