[Webkit-unassigned] [Bug 80542] [Qt WK2] Disable/enable mouse events when displaying dialogs only for desktop view

[bugzilla-daemon at webkit.org]

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





--- Comment #9 from Dinu Jacob <dinu.jacob at Nokia.com>  2012-03-20 07:37:12 PST ---
(In reply to comment #8)
> (In reply to comment #7)
> > As flickable webView wouldn't receive the mouse events is there any benefit of enabling mouse events for it?
> 
> Sorry, I was somewhat superficial when reading your questions and did not look at the patch.
> 
> If the Flickable is in use the mouse handling on the WebView has to be disabled since they will be delivered to the Flickable first, which is undesirable.
> This is another reason why we need to subclass Flickable for the WebView item instead of internally instantiating it.
> 
> Qt only synthesizes mouse events for touch events which were not accepted, so if the touch event sent by the touch device would be accepted we would not receive a mouse event from Qt.
> 
> MiniBrowser and the WebView is indeed not yet prepared for the mentioned hybrid devices and fixing this is clearly not the scope of this bug.
> 
> For the currently supported configurations we should keep the distinction between the touch and desktop view, hence I think this patch is correct and fixes inconsistent behavior of the web view.

Simon, can you take a look at this again?

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