Bug 152920 - CSS3 columns - invisible elements in non-first column
Summary: CSS3 columns - invisible elements in non-first column
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: Other
Hardware: All All
: P2 Normal
Assignee: Dave Hyatt
URL: https://sim642.github.io/FLL-scorer/
Keywords: InRadar
: 155823 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-01-08 13:04 PST by Simmo Saan
Modified: 2016-03-23 18:56 PDT (History)
8 users (show)

See Also:

Attachments
Screenshot of the page in question (207.76 KB, image/png)
2016-01-08 13:09 PST, Simmo Saan 		no flags Details
Patch (7.21 KB, patch)
2016-01-21 10:47 PST, Dave Hyatt 		simon.fraser: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Simmo Saan 2016-01-08 13:04:40 PST 
I have a bootstrap page in which I make use of CSS3 columns: https://sim642.github.io/FLL-scorer/. These panels contain bootstrap radio button groups.

When viewing this page on a wide enough screen to cause it to activate the multi-column layout, the button groups turn invisible but still exist in inspector. When forcing a single column layout they show up as intended. On iOS Safari I noticed them being broken a bit differently: some of the panel content would show up fully, some would be invisible, even the text.

The problem is that elements seem to randomly turn invisible in columns which are not the first.
Comment 1 Simmo Saan 2016-01-08 13:09:37 PST 
Created attachment 268574 [details]
Screenshot of the page in question
Comment 2 Radar WebKit Bug Importer 2016-01-08 13:09:56 PST 
<rdar://problem/24111946>
Comment 3 Dave Hyatt 2016-01-21 10:47:14 PST 
Created attachment 269468 [details]
Patch
Comment 4 Dean Jackson 2016-01-21 10:52:00 PST 
Comment on attachment 269468 [details]
Patch

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

> Source/WebCore/ChangeLog:19
> +        Make sure the crawl up the containing block chain to apply clips properly offsets
> +        to account for columns. convertToLayerCoords could already handle this, so
> +        offsetFromAncestor now takes the same extra argument (whether or not to adjust for
> +        columns) that convertToLayerCoords does.

This bit usually goes before the list of files.

> LayoutTests/fast/multicol/border-radius-overflow-columns.html:12
> +            -webkit-columns: 2;

Don't need this any more.
Comment 5 Dave Hyatt 2016-01-22 09:36:16 PST 
Fixed in r195453.
Comment 6 Simon Fraser (smfr) 2016-03-23 18:56:59 PDT 
*** Bug 155823 has been marked as a duplicate of this bug. ***