diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-10-11 14:23:04 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2021-10-11 18:21:49 +0200 |
commit | 83cadf937ff9a0fd4ca24627673d45d1bab8c914 (patch) | |
tree | 054a618d7bb37d2343e65a75d2a730dee2628a10 | |
parent | 8fb4889840a6c79f6bad7bb7a322d86b751cdd93 (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.cxx | 2 |
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; } } |