From 252cf479c483022505efc2641b64fa7cfde385b1 Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Wed, 14 Oct 2020 16:42:26 +0100 Subject: tdf#137294 don't change subset sensitivity after setting search mode Change-Id: Iaa76e2bc8029ac8f8a52b0c45f74b57395090088 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104307 Tested-by: Jenkins Reviewed-by: Michael Stahl --- cui/source/dialogs/cuicharmap.cxx | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'cui') diff --git a/cui/source/dialogs/cuicharmap.cxx b/cui/source/dialogs/cuicharmap.cxx index 13d3c8e7658b..bcc1dd16a454 100644 --- a/cui/source/dialogs/cuicharmap.cxx +++ b/cui/source/dialogs/cuicharmap.cxx @@ -643,11 +643,6 @@ IMPL_LINK_NOARG(SvxCharacterMap, FontSelectHdl, weld::ComboBox&, void) m_xShowSet->SetFont( aFont ); m_xSearchSet->SetFont( aFont ); m_aShowChar.SetFont( aFont ); - if (isSearchMode) - { - SearchUpdateHdl(*m_xSearchText); - SearchCharHighlightHdl(m_xSearchSet.get()); - } // setup unicode subset listbar with font specific subsets, // hide unicode subset listbar for symbol fonts @@ -675,6 +670,14 @@ IMPL_LINK_NOARG(SvxCharacterMap, FontSelectHdl, weld::ComboBox&, void) m_xSubsetText->set_sensitive(bNeedSubset); m_xSubsetLB->set_sensitive(bNeedSubset); + if (isSearchMode) + { + // tdf#137294 do this after modifying m_xSubsetLB sensitivity to + // restore insensitive for the search case + SearchUpdateHdl(*m_xSearchText); + SearchCharHighlightHdl(m_xSearchSet.get()); + } + // tdf#118304 reselect current glyph to see if its still there in new font selectCharByCode(Radix::hexadecimal); } -- cgit