diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-10-14 16:42:26 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-10-14 20:25:42 +0200 |
| commit | d7ff5df1dc5c7ae0f468e0250c54e9f72035093c (patch) | |
| tree | 192084f06f1d405cf1fed615cae9982566ad1fcd | |
| parent | b341758a6df28d2c2034fd2335924890ccb2b905 (diff) | |
tdf#137294 don't change subset sensitivity after setting search mode
Change-Id: Iaa76e2bc8029ac8f8a52b0c45f74b57395090088
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104312
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
| -rw-r--r-- | cui/source/dialogs/cuicharmap.cxx | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/cui/source/dialogs/cuicharmap.cxx b/cui/source/dialogs/cuicharmap.cxx index 25b05c68d2b1..f78b70a29662 100644 --- a/cui/source/dialogs/cuicharmap.cxx +++ b/cui/source/dialogs/cuicharmap.cxx @@ -644,11 +644,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 @@ -676,6 +671,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); } |