Since Grid Layout doe snot support orthogonal flows, the alignment logic lacks support for that cases as well.
Created attachment 284214 [details] Patch
Comment on attachment 284214 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=284214&action=review > Source/WebCore/rendering/RenderGrid.cpp:2102 > + // TODO (lajava): Should we implement this logic in a generic utility function ? Looks like a comment using some other project’s competing standards. Maybe Chromium? > Source/WebCore/rendering/RenderGrid.cpp:2115 > + // TODO (lajava): Should we implement this logic in a generic utility function ? Ditto. > Source/WebCore/rendering/RenderGrid.cpp:2167 > + // TODO (lajava): Should we implement this logic in a generic utility function ? Ditto. > Source/WebCore/rendering/RenderGrid.cpp:2180 > + // TODO (lajava): Should we implement this logic in a generic utility function ? Ditto.
Created attachment 284568 [details] Patch Applied suggested changes.
Created attachment 284584 [details] Patch Patch rebased.
Comment on attachment 284584 [details] Patch Clearing flags on attachment: 284584 Committed r203771: <http://trac.webkit.org/changeset/203771>
All reviewed patches have been landed. Closing bug.
*** Bug 107738 has been marked as a duplicate of this bug. ***