New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 607597 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
NOT IN USE
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Inconsistency with computing scrollable bounds for multicolumn specific values

Reported by bbri...@microsoft.com, Apr 28 2016

Issue description

We've investigated interop problems with redfin.com mobile site and our developer reduced this to the following problem:

1. Multi-column with column width = 16px doesn't make overflowing container scrollable - https://jsfiddle.net/pL25ex60/1/

2. Multi-column with column width = 17px (and I believe everything that is not 16px) makes overflowing container scrollable - https://jsfiddle.net/pL25ex60/

Doesn't repro in Edge, Firefox. Currently makes redfin unusable on anything but Android (didn't try iOS).


 

Comment 1 by msten...@opera.com, Apr 28 2016

Owner: msten...@opera.com
Status: Assigned (was: Untriaged)
The problem occurs when the multicol container width is less than or equal to column-gap (which is 16px in the test).
Project Member

Comment 2 by bugdroid1@chromium.org, May 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/46be82a9fa3fdbe05e2f8137ee61a2d3f734cab4

commit 46be82a9fa3fdbe05e2f8137ee61a2d3f734cab4
Author: mstensho <mstensho@opera.com>
Date: Wed May 04 11:01:13 2016

Improve multicol overflow rect calculation when column width is 0.

If column-gap is larger than the content box width of the multicol container,
we'll end up with zero-width column boxes (that are allowed to overflow into
neighboring gaps).

BUG= 607597 

Review-Url: https://codereview.chromium.org/1927283002
Cr-Commit-Position: refs/heads/master@{#391477}

[add] https://crrev.com/46be82a9fa3fdbe05e2f8137ee61a2d3f734cab4/third_party/WebKit/LayoutTests/fast/multicol/overflowing-columns-large-gaps-expected.txt
[add] https://crrev.com/46be82a9fa3fdbe05e2f8137ee61a2d3f734cab4/third_party/WebKit/LayoutTests/fast/multicol/overflowing-columns-large-gaps.html
[modify] https://crrev.com/46be82a9fa3fdbe05e2f8137ee61a2d3f734cab4/third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.cpp

Comment 3 by msten...@opera.com, May 4 2016

Status: Fixed (was: Assigned)
Verified fixed at the very least on the provided repro case. Thank you!

Sign in to add a comment