35834 2010-03-06 14:44:05 -0800 [Qt] QGraphicsWebView: Arrow keys scroll the graphics-view instead of the web-page 2010-04-27 14:37:32 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) PC Linux RESOLVED FIXED http://www.youtube.com/xl Qt P1 Minor --- 1 noam webkit-qt-unassigned commit-queue jturcotte oldest_to_newest 196940 0 noam 2010-03-06 14:44:05 -0800 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. 203015 1 jturcotte 2010-03-23 06:15:27 -0700 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. 217669 2 54412 benjamin 2010-04-27 06:15:50 -0700 Created attachment 54412 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. 217676 3 54412 kenneth 2010-04-27 06:47:26 -0700 Comment on attachment 54412 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. 217849 4 54412 commit-queue 2010-04-27 12:34:09 -0700 Comment on attachment 54412 Patch Clearing flags on attachment: 54412 Committed r58321: <http://trac.webkit.org/changeset/58321> 217850 5 commit-queue 2010-04-27 12:34:14 -0700 All reviewed patches have been landed. Closing bug. 217953 6 noam 2010-04-27 14:37:32 -0700 So a developer implementing an application with QGraphicsWebView and keypad navigation would have to do this sceneRect step each time? 54412 2010-04-27 06:15:50 -0700 2010-04-27 12:34:09 -0700 Patch 35834.patch text/plain 1194 benjamin ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv ZwppbmRleCA4MjA0YTdjLi4wY2Q3MGRlIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDQt MjcgIEJlbmphbWluIFBvdWxhaW4gIDxiZW5qYW1pbi5wb3VsYWluQG5va2lhLmNvbT4KKworICAg ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbUXRdIFFHcmFwaGlj c1dlYlZpZXc6IEFycm93IGtleXMgc2Nyb2xsIHRoZSBncmFwaGljcy12aWV3IGluc3RlYWQgb2Yg dGhlIHdlYi1wYWdlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn aT9pZD0zNTgzNAorCisgICAgICAgIFRoZSBzY2VuZSBzaG91bGQgYWx3YXlzIGhhdmUgdGhlIHNp emUgb2YgdGhlIHdlYiB2aWV3IG90aGVyd2hpc2UgaXQgaXMKKyAgICAgICAgcG9zc2libGUgdG8g c2Nyb2xsIHRoZSBncmFwaGljcyB2aWV3LgorCisgICAgICAgICogUXRMYXVuY2hlci93ZWJ2aWV3 LmNwcDoKKyAgICAgICAgKFdlYlZpZXdHcmFwaGljc0Jhc2VkOjpyZXNpemVFdmVudCk6CisKIDIw MTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZp ZXdlZCBieSBNYWNpZWogU3RhY2hvd2lhay4KZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1F0TGF1 bmNoZXIvd2Vidmlldy5jcHAgYi9XZWJLaXRUb29scy9RdExhdW5jaGVyL3dlYnZpZXcuY3BwCmlu ZGV4IGJlZDgxNWUuLjE0NjdjZjggMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1F0TGF1bmNoZXIv d2Vidmlldy5jcHAKKysrIGIvV2ViS2l0VG9vbHMvUXRMYXVuY2hlci93ZWJ2aWV3LmNwcApAQCAt OTksNiArOTksNyBAQCB2b2lkIFdlYlZpZXdHcmFwaGljc0Jhc2VkOjpyZXNpemVFdmVudChRUmVz aXplRXZlbnQqIGV2ZW50KQogICAgICAgICByZXR1cm47CiAgICAgUVJlY3RGIHJlY3QoUVBvaW50 KDAsIDApLCBldmVudC0+c2l6ZSgpKTsKICAgICBtX2l0ZW0tPnNldEdlb21ldHJ5KHJlY3QpOwor ICAgIHNjZW5lKCktPnNldFNjZW5lUmVjdChyZWN0KTsKIH0KIAogdm9pZCBXZWJWaWV3R3JhcGhp Y3NCYXNlZDo6c2V0RnJhbWVSYXRlTWVhc3VyZW1lbnRFbmFibGVkKGJvb2wgZW5hYmxlZCkK