[Webkit-unassigned] [Bug 70898] Range sliders and spin buttons don't work with multi-columns

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 31 03:52:55 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=70898





--- Comment #7 from yosin at chromium.org  2011-10-31 03:52:55 PST ---
adjustColumns expects logical column coordinate rather than physical column coordinate, terminology may not be common, see below for logical/physical column coordinate:

Physical column Coordinate
+-------+-------+-------+
|       |       |   x   |
+-------+-------+-------+

Logical Column Coordinate
+-------+
|       |
+-------+
|       |
+-------+
|   x   |
+-------+

So, we can't pass actual point to offsetFromContainer and this patch doesn't replicate code of adjustColumns.
// I guess this was reason why original code pass LayoutPoint() instead of passing actual point in mapLocalToContainer.

Pass (0, 0) to offsetFromContainer calculates offset with considering progressive axis.

So, I think this patch does right thing.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list