Closed
Bug 71339
Opened 23 years ago
Closed 23 years ago
check in bidi code for layout/html/forms
Categories
(Core :: Layout, defect, P1)
Core
Layout
Tracking
()
VERIFIED
FIXED
mozilla0.9
People
(Reporter: erik, Assigned: erik)
Details
Attachments
(2 files)
16.79 KB,
patch
|
Details | Diff | Splinter Review | |
14.59 KB,
patch
|
Details | Diff | Splinter Review |
This bug report will be used to attach the diffs for layout/html/forms from the IBM bidi project.
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Comment 2•23 years ago
|
||
Rod, please (have someone) review these changes. Thanks!
Comment 3•23 years ago
|
||
Wow, a lot of changes, I'll apply the patch and take a look.
Assignee | ||
Comment 4•23 years ago
|
||
Rod, how's it going? Note that the changes are wrapped in #ifdef IBMBIDI, and we won't be turning on the ifdef until later, so you shouldn't see anything when you apply these patches.
Assignee | ||
Updated•23 years ago
|
Priority: -- → P1
Target Milestone: --- → mozilla0.8.1
Comment 5•23 years ago
|
||
r=rods since they aren't turned on go ahead and check them in. But we should talk before they get turned on.
Assignee | ||
Comment 6•23 years ago
|
||
Assignee | ||
Comment 8•23 years ago
|
||
Changes checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•23 years ago
|
||
Rod, we are still reviewing and checking in #ifdef IBMBIDI changes in other parts of the tree, but we would like to turn on the ifdef soon afterwards, so would you please let me know what other concerns you might have with this part (i.e. forms)? Let me know whether it's OK to turn on the ifdef for forms. Thx.
Comment 10•23 years ago
|
||
code change bug, reassign QA contact to ftang v=ftang
Status: RESOLVED → VERIFIED
QA Contact: petersen → ftang
You need to log in
before you can comment on or make changes to this bug.
Description
•