extensions/source/propctrlr/browserline.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 83cadf937ff9a0fd4ca24627673d45d1bab8c914
Author:     Caolán McNamara <[email protected]>
AuthorDate: Mon Oct 11 14:23:04 2021 +0100
Commit:     Caolán McNamara <[email protected]>
CommitDate: Mon Oct 11 18:21:49 2021 +0200

    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 <[email protected]>
    Reviewed-by: Caolán McNamara <[email protected]>

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;
             }
         }

Reply via email to