Comment 8 for bug 1599771

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

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