diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-09-27 20:06:17 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2020-10-01 09:55:35 +0200 |
commit | 9d30d3bb93171a12655dc89066fe065e2c2af658 (patch) | |
tree | 2c8f841250f4b6d7a92adf2e5ecd4979962940d1 /svtools | |
parent | 77960eb573e548816064690c5b1de5ca18ae9cbd (diff) |
Related: tdf#137016 check if subcontrol has the focus
backport some required pieces from master to support that
Change-Id: I632188bc0512c9d8935bd0898c96e066881ebeb1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103524
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'svtools')
-rw-r--r-- | svtools/source/brwbox/ebbcontrols.cxx | 8 | ||||
-rw-r--r-- | svtools/source/brwbox/editbrowsebox.cxx | 13 | ||||
-rw-r--r-- | svtools/source/brwbox/editbrowsebox2.cxx | 2 |
3 files changed, 15 insertions, 8 deletions
diff --git a/svtools/source/brwbox/ebbcontrols.cxx b/svtools/source/brwbox/ebbcontrols.cxx index 1e0272aeeea9..e402f0a42a40 100644 --- a/svtools/source/brwbox/ebbcontrols.cxx +++ b/svtools/source/brwbox/ebbcontrols.cxx @@ -28,7 +28,7 @@ namespace svt //= ComboBoxControl ComboBoxControl::ComboBoxControl(vcl::Window* pParent) - : InterimItemWindow(pParent, "svt/ui/combocontrol.ui", "ComboControl") + : ControlBase(pParent, "svt/ui/combocontrol.ui", "ComboControl") , m_xWidget(m_xBuilder->weld_combo_box("combobox")) { m_xWidget->set_entry_width_chars(1); // so a smaller than default width can be used @@ -38,7 +38,7 @@ namespace svt void ComboBoxControl::dispose() { m_xWidget.reset(); - InterimItemWindow::dispose(); + ControlBase::dispose(); } IMPL_LINK_NOARG(ComboBoxControl, SelectHdl, weld::ComboBox&, void) @@ -111,7 +111,7 @@ namespace svt //= ListBoxControl ListBoxControl::ListBoxControl(vcl::Window* pParent) - : InterimItemWindow(pParent, "svt/ui/listcontrol.ui", "ListControl") + : ControlBase(pParent, "svt/ui/listcontrol.ui", "ListControl") , m_xWidget(m_xBuilder->weld_combo_box("listbox")) { m_xWidget->set_size_request(42, -1); // so a later narrow size request can stick @@ -121,7 +121,7 @@ namespace svt void ListBoxControl::dispose() { m_xWidget.reset(); - InterimItemWindow::dispose(); + ControlBase::dispose(); } IMPL_LINK_NOARG(ListBoxControl, SelectHdl, weld::ComboBox&, void) diff --git a/svtools/source/brwbox/editbrowsebox.cxx b/svtools/source/brwbox/editbrowsebox.cxx index 0105082cea5c..12ef6d6fd181 100644 --- a/svtools/source/brwbox/editbrowsebox.cxx +++ b/svtools/source/brwbox/editbrowsebox.cxx @@ -211,7 +211,7 @@ namespace svt if (IsEditing()) { EnableAndShow(); - if (!aController->GetWindow().HasFocus() && (m_pFocusWhileRequest.get() == Application::GetFocusWindow())) + if (!ControlHasFocus() && (m_pFocusWhileRequest.get() == Application::GetFocusWindow())) aController->GetWindow().GrabFocus(); } } @@ -467,6 +467,13 @@ namespace svt implActivateCellOnMouseEvent(rEvt, true); } + bool EditBrowseBox::ControlHasFocus() const + { + Window* pControlWindow = aController ? &aController->GetWindow() : nullptr; + if (ControlBase* pControlBase = dynamic_cast<ControlBase*>(pControlWindow)) + return pControlBase->ControlHasFocus(); + return pControlWindow && pControlWindow->HasChildPathFocus(); + } void EditBrowseBox::implActivateCellOnMouseEvent(const BrowserMouseEvent& _rEvt, bool _bUp) { @@ -474,7 +481,7 @@ namespace svt ActivateCell(); else if (IsEditing() && !aController->GetWindow().IsEnabled()) DeactivateCell(); - else if (IsEditing() && !aController->GetWindow().HasChildPathFocus()) + else if (IsEditing() && !ControlHasFocus()) AsynchGetFocus(); if (!(IsEditing() && aController->GetWindow().IsEnabled() && aController->WantMouseEvent())) @@ -546,7 +553,7 @@ namespace svt { if (rEvt.GetType() == MouseNotifyEvent::KEYINPUT) { - if ( (IsEditing() && Controller()->GetWindow().HasChildPathFocus()) + if ( (IsEditing() && ControlHasFocus()) || rEvt.GetWindow() == &GetDataWindow() || (!IsEditing() && HasChildPathFocus()) ) diff --git a/svtools/source/brwbox/editbrowsebox2.cxx b/svtools/source/brwbox/editbrowsebox2.cxx index 3544979e725b..7c84cf5eb805 100644 --- a/svtools/source/brwbox/editbrowsebox2.cxx +++ b/svtools/source/brwbox/editbrowsebox2.cxx @@ -124,7 +124,7 @@ void EditBrowseBox::GrabTableFocus() void EditBrowseBox::DetermineFocus( const GetFocusFlags _nGetFocusFlags ) { - bool bFocus = false; + bool bFocus = ControlHasFocus(); for (vcl::Window* pWindow = Application::GetFocusWindow(); pWindow && !bFocus; pWindow = pWindow->GetParent()) |