[Webkit-unassigned] [Bug 183889] [iOS] Make the Find UI better estimate the obscured area when revealing highlighted text

[bugzilla-daemon at webkit.org]

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

--- Comment #2 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
I took a look into this. As I understand the logic of adjustScrollStepForFixedContent is to check if some fixed opaque content covers the full width at the top/bottom of the frame so that vertical page scroll can be adjusted appropriately on macOS.

I guess we can re-use this logic but we have couple differences here. First since fixed content is relative to parent subframe, adjusting the page scroll of "overflow: auto" node does not seem to work in general (we would have to scroll the parent subframe to make the overflow node visible). Second, I'm not exactly sure page scroll makes sense on iOS and anyway here it's not page scroll but "scroll to reveal match" which ends up at scrollRectToVisible.

Finally, in the AMP use case of bug 183658, the frame area is essentially the same as the "overflow: auto" area so we could apply the same logic but I wonder whether we should rather focus on making frame scrollable on iOS (bug 149264) in order to address these kinds of use case of "scrollable page with fixed-positioned header/footer".

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180327/e0c1f711/attachment.html>