diff options
| author | Jüri Valdmann <juri.valdmann@qt.io> | 2018-05-15 11:51:16 +0200 |
|---|---|---|
| committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2018-05-15 20:05:20 +0000 |
| commit | 694350898f4d96a57a1f91e36519070ed4b4c09b (patch) | |
| tree | 9561bff5bb1b95c35594325ef0aeac914b9d1524 /src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp | |
| parent | ce01bf795b5a50e28e119d1925b395b8bfcf419b (diff) | |
Fix keyboard focus for speculative RenderFrameHosts
When encountering a 301 redirect, one render frame/widget is created for the
original URL plus one "speculative" render frame/widget for the new URL. Once
the speculative frame commits, keyboard focus should switch to the corresponding
widget. This doesn't work however, because QQuickItem::forceActiveFocus doesn't
give focus to the containing QQuickWidget. Fixed by using QWidget::setFocus.
Also changed simplebrowser to focus the QWebEngineView on startup.
Task-number: QTBUG-68076
Change-Id: I8dc42ba89bfdcd46a86c7dca357fdf1e94f439d4
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp')
| -rw-r--r-- | src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp b/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp index 56f04d424..98482ae78 100644 --- a/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp +++ b/src/webenginewidgets/render_widget_host_view_qt_delegate_widget.cpp @@ -253,6 +253,7 @@ void RenderWidgetHostViewQtDelegateWidget::setKeyboardFocus() QGuiApplication::sync(); m_rootItem->forceActiveFocus(); + setFocus(); } bool RenderWidgetHostViewQtDelegateWidget::hasKeyboardFocus() |