Bug 141207 - Tables don't repaginate properly when pagination height changes
Summary: Tables don't repaginate properly when pagination height changes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dave Hyatt
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-02-03 12:51 PST by Dave Hyatt
Modified: 2015-02-04 13:39 PST (History)
6 users (show)

See Also:


Attachments
Patch (13.88 KB, patch)
2015-02-03 13:03 PST, Dave Hyatt
dino: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Hyatt 2015-02-03 12:51:49 PST
Tables don't repaginate properly when the pagination height changes or if the offset of the table changes on the page. This is because sections need to be marked for layout whenever a table gets marked for layout.
Comment 1 Dave Hyatt 2015-02-03 13:03:50 PST
Created attachment 245958 [details]
Patch
Comment 2 Dean Jackson 2015-02-03 13:06:40 PST
Comment on attachment 245958 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=245958&action=review

> LayoutTests/fast/multicol/table-dynamic-movement.html:8
> +<script>
> +function doIt()
> +{
> +document.getElementById('test').offsetHeight
> +document.getElementById('test').style.height = '500px';
> +}
> +</script>

I think you could have put this directly inline.....

> LayoutTests/fast/multicol/table-dynamic-movement.html:21
> +<script>
> +doIt()
> +</script>

.... down here.
Comment 3 Jon Lee 2015-02-04 13:19:42 PST
rdar://problem/18387659
Comment 4 Dave Hyatt 2015-02-04 13:39:32 PST
Landed in r179627.