I think I'd prefer to just use QQuickWindow::activeFocusItem in
oxide::qquick::ContentsView::HasFocus(). From looking at the code, I
don't even think that QQuickItem::hasActiveFocus has the correct
semantics as it will indicate active focus for ancestors of the active
focus item which isn't really what we want (ie, if an application adds
another item as a child of the webview, we shouldn't think we're
actively focused if the child item is the active focus item).

We should really add some integration tests for these cases in
qt/tests/qmltests/web_platform

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1599771

Title:
  Webview appears to think it's in focus when typing in the addressbar

To manage notifications about this bug go to:
https://bugs.launchpad.net/oxide/+bug/1599771/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to