summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2021-10-11 14:23:04 +0100
committerCaolán McNamara <caolanm@redhat.com>2021-10-11 18:21:49 +0200
commit83cadf937ff9a0fd4ca24627673d45d1bab8c914 (patch)
tree054a618d7bb37d2343e65a75d2a730dee2628a10
parent8fb4889840a6c79f6bad7bb7a322d86b751cdd93 (diff)
tdf#145033 remove old focus handler so no double-connect attempt is made
doesn't hurt in this case, but we assert because it has been a fairly common problem in the past Change-Id: Id312c8d2693df803962c265a9a49926d8df04cc0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123413 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r--extensions/source/propctrlr/browserline.cxx2
1 files changed, 2 insertions, 0 deletions
diff --git a/extensions/source/propctrlr/browserline.cxx b/extensions/source/propctrlr/browserline.cxx
index 8618ecffc8fe..d4770ba3e5d1 100644
--- a/extensions/source/propctrlr/browserline.cxx
+++ b/extensions/source/propctrlr/browserline.cxx
@@ -346,6 +346,7 @@ namespace pcr
if (m_pBrowseButton)
{
m_pBrowseButton->hide();
+ m_pBrowseButton->connect_focus_in(Link<weld::Widget&, void>());
m_pBrowseButton = nullptr;
}
}
@@ -354,6 +355,7 @@ namespace pcr
if (m_pAdditionalBrowseButton)
{
m_pAdditionalBrowseButton->hide();
+ m_pAdditionalBrowseButton->connect_focus_in(Link<weld::Widget&, void>());
m_pAdditionalBrowseButton = nullptr;
}
}