RESOLVED FIXED 103579
Positioned replaced elements should resolve vertical margins against their containing block's logical width
https://bugs.webkit.org/show_bug.cgi?id=103579
Summary Positioned replaced elements should resolve vertical margins against their co...
Bear Travis
Reported 2012-11-28 17:01:13 PST
They currently use the element's block-direction measurement of the containing block.
Attachments
Initial patch (8.20 KB, patch)
2012-11-29 17:36 PST, Bear Travis
no flags
Updated patch (10.90 KB, patch)
2012-11-29 17:39 PST, Bear Travis
no flags
Updating tests (16.26 KB, patch)
2012-11-29 18:02 PST, Bear Travis
no flags
Updating changelogs & tests (14.42 KB, patch)
2012-12-04 12:42 PST, Bear Travis
no flags
Converting to static html content (14.29 KB, patch)
2012-12-13 17:19 PST, Bear Travis
no flags
Bear Travis
Comment 1 2012-11-29 17:36:37 PST
Created attachment 176861 [details] Initial patch First pass at a patch, using the containing block's logical width to resolve all percentage margins.
Bear Travis
Comment 2 2012-11-29 17:39:16 PST
Created attachment 176863 [details] Updated patch Uploading a newer version of the patch, with changelogs and tests.
Bear Travis
Comment 3 2012-11-29 18:02:10 PST
Created attachment 176871 [details] Updating tests
Bear Travis
Comment 4 2012-12-04 12:42:14 PST
Created attachment 177529 [details] Updating changelogs & tests
Bear Travis
Comment 5 2012-12-13 17:19:36 PST
Created attachment 179384 [details] Converting to static html content Creating the containers and elements with HTML markup rather than Javascript to make the test easier to read.
Emil A Eklund
Comment 6 2012-12-13 17:47:30 PST
Comment on attachment 179384 [details] Converting to static html content Thanks for making the test changes I suggested. Looks good.
WebKit Review Bot
Comment 7 2012-12-13 18:14:35 PST
Comment on attachment 179384 [details] Converting to static html content Clearing flags on attachment: 179384 Committed r137695: <http://trac.webkit.org/changeset/137695>
WebKit Review Bot
Comment 8 2012-12-13 18:14:39 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.