Summary: | Navigating downwards / upwards does not focus on the links spread across more than one line. | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | deepak <deepak.dhananjaya> | ||||||||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Blocker | CC: | ademar, suresh.voruganti, tonikitoo, yael | ||||||||||
Priority: | P1 | Keywords: | Qt | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | S60 Hardware | ||||||||||||
OS: | S60 3rd edition | ||||||||||||
Attachments: |
|
Description
deepak
2011-02-17 05:06:42 PST
Additional Info: Tested this with QTTestBrowser, and it is reproducible on the same. Additional Info: Since QTTestBrowser does not support key pad navigation, enabled the setting through code in webpage. this->settings()->setAttribute(QWebSettings::SpatialNavigationEnabled, true); After which, was able to test this test case. (In reply to comment #0) > Attachment : includes a webpage, that has link that is spread across two lines. > > Steps: > 1. Open the page, and navigate use key pad in the mobile hardware to navigate the links. > > Expected Result: > The link that is spread across two lines also should be selected and focused. > > Actual REsult: > The link that is spread across two lines is skipped and the next link is selected and focused. Please include the attachment. Created attachment 82814 [details]
widget
Created attachment 82820 [details]
Test page attached.
Removing [Qt][S60] as the fix is in common code. Patch is coming soon. Created attachment 82946 [details]
Patch.
When 2 anchor elements span more than one line each, and one ends on the same line that the
second starts on, the rects reported by their renderers are overlapping. When handling 2 overlapping nodes, check for this case, and don't assume that one of the nodes is on a higher layer.
Comment on attachment 82946 [details]
Patch.
Thanks for the review :)
Committed r79021: <http://trac.webkit.org/changeset/79021> Please cherry pick the fix for Qtwebkit 2.1 (In reply to comment #10) > Please cherry pick the fix for Qtwebkit 2.1 The code requires significant changes to be cherry-picked. We need a backport targeted at qtwebkit-2.1. Created attachment 83393 [details]
Backport
This is a backport of the original patch to qtwebkit-2.1.x
Revision r79021 cherry-picked into qtwebkit-2.1 with commit 3721aa1 <http://gitorious.org/webkit/qtwebkit/commit/3721aa1> |