Closed Bug 558574 Opened 10 years ago Closed 10 years ago

tfoot placement wrong (print preview, print)

Categories

(Core :: Layout: Tables, defect)

x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: bernd_mozilla, Assigned: bernd_mozilla)

References

Details

(Keywords: testcase)

Attachments

(8 files)

Attached file reftest
see attached test case, the expected rendering is in the reference
Attached file reference
compare the reftest in IE print preview how it should be rendered
Summary: tfoot placement wrong → tfoot placement wrong (print preview, print)
look in print preview,  this executes the code 

http://mxr.mozilla.org/mozilla-central/source/layout/tables/nsTableFrame.cpp#2765
where roc's comment is exactly to the point.
it also fails.
Attached patch patchSplinter Review
Comment on attachment 443037 [details] [diff] [review]
patch

Robert this are the bugs that I fixed while writing some reftests for table pagination.

I pushed this already over the try server
Attachment #443037 - Flags: review?(roc)
Blocks: 558575
The tests are in bug 558575
Assignee: nobody → bernd_mozilla
Comment on attachment 443037 [details] [diff] [review]
patch

+  PRBool InternalBreakBefore() const;
+  PRBool InternalBreakAfter() const;

Make these "HasInternalBreak*" so they don't look like verbs.
Attachment #443037 - Flags: review?(roc) → review+
http://hg.mozilla.org/mozilla-central/rev/fcff7673a6ae
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Depends on: 567835
You need to log in before you can comment on or make changes to this bug.