Skip to content
This repository has been archived by the owner on Jan 22, 2022. It is now read-only.

Opening a context menu seems to cause morph-browser to lose focus #192

Closed
JulianGro opened this issue Jun 7, 2019 · 4 comments · Fixed by #220
Closed

Opening a context menu seems to cause morph-browser to lose focus #192

JulianGro opened this issue Jun 7, 2019 · 4 comments · Fixed by #220
Labels
bug Something isn't working

Comments

@JulianGro
Copy link

Opening a context menu seems to cause morph-browser to lose focus.

Steps to reproduce:
Go to a website with a link.
Open the context menu on the link by holding your finger on it
Press any button in the context menu
You will now have to press on the browser before you can scroll or open a new context menu again.

What should happen:
You should be able to scroll or click directly after closing a context menu.

@balcy balcy added the bug Something isn't working label Jun 8, 2019
@balcy
Copy link
Collaborator

balcy commented Aug 3, 2019

what I've found out so far is, that the error seems to be already present in QtWebEngine itself.

If in WebViewImpl.qml the function onContextMenuRequested: function(request) is removed, or return; is put as its first line, you still have the effect that you need to click once on the browser, and only then it is possible to scroll again.

It is like the "real" / native context menu, which is not visible, took away the focus and is closed upon the click. [edit: speculation]

So it is not related to the dialogs / context menus shown by the morph browser.

@balcy
Copy link
Collaborator

balcy commented Aug 19, 2019

ok, I think the real cause was code in MorphWebView that was catching onContextMenuRequested as well.

@ziggutas
Copy link

BQ E4.5 rc (2019-W41/2)
I cannot reproduce this issue

@mgns-gh
Copy link

mgns-gh commented Oct 12, 2019

FP2 RC (2019-W41/2)
The Context menu seems to work as expected.
I cannot reproduce the issue described.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants