Bug 35834 – [Qt] QGraphicsWebView: Arrow keys scroll the graphics-view instead of the web-page

• Component: New Bugs
• Status: RESOLVED FIXED

[Noam Rosenthal]

To reproduce:
open the above website:
- QtLauncher -graphicsbased -url http://www.youtube.com/xl
- Resize the QtLauncher window to be a little smaller
- Press the down key several times

While the keys should affect the list selection, they actually move around the whole QGraphicsWebView.
This does not happen with QWebView (no -graphicsbased), hence the assumption that the bug is in the "Webkit Qt" module

I tested on Ubuntu 9.04, but I believe the problem exists on all Qt platforms.

[Jocelyn Turcotte]

Reproduced on Windows as well (though its a bit harder than on linux).

I could also notice that it happens more often if you play with the tab switching.

QtLauncher only have the address bar and the web view that takes the focus, however, sometimes, the focus gets taken by another widget between the two (so it sometimes takes 2 tabs to switch the focus from the address to the web view). The bug happens when you press the arrow keys while focused on this intermediate widget.

If you don't resize the window a bit, you still get the focus steal problem, but you won't see the "move around the whole QGraphicsWebView" problem.

According to Noam this bug is an annoyance for many embedded developers.
Setting to P1-minor since I think that this is worth fixing it for the next release.

[Benjamin Poulain]

Created attachment 54412 [details]
Patch

QGraphicsView do not do anything wrong. It does not use the event so it ends up in the scroll area code.

We just have a make sure the scene has the correct size when we resize the graphicswebview.

[Kenneth Rohde Christiansen]

Comment on attachment 54412 [details]
Patch

I managed to trick a similar bug using wheel and opening a combobox and it scrolled the graphics view instead of the page and left behind garbage.

[WebKit Commit Bot]

Comment on attachment 54412 [details]
Patch

Clearing flags on attachment: 54412

Committed r58321: <http://trac.webkit.org/changeset/58321>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Noam Rosenthal]

So a developer implementing an application with QGraphicsWebView and keypad navigation would have to do this sceneRect step each time?