diff options
author | Eike Rathke <erack@redhat.com> | 2015-02-12 23:47:21 +0100 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2015-02-13 00:51:10 +0100 |
commit | a448f1bb9221f30f73574c580b506e00bf2ae61a (patch) | |
tree | 4a41b88334f305eace166f5a4619f579a1024891 /svx/source | |
parent | ba686b9bd2596811141e4028947334f10799c356 (diff) |
Resolves: tdf#81429 select corresponding listbox entry if not current
Change-Id: Ib475a43601ee22d33677ac45cd6ef9dba7e7c431
Diffstat (limited to 'svx/source')
-rw-r--r-- | svx/source/dialog/langbox.cxx | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/svx/source/dialog/langbox.cxx b/svx/source/dialog/langbox.cxx index 1f46a8683172..8b31ee4cd78c 100644 --- a/svx/source/dialog/langbox.cxx +++ b/svx/source/dialog/langbox.cxx @@ -713,21 +713,29 @@ IMPL_LINK( SvxLanguageComboBox, EditModifyHdl, SvxLanguageComboBox*, /*pEd*/ ) const sal_Int32 nPos = GetEntryPos( aStr); if (nPos != COMBOBOX_ENTRY_NOTFOUND) { - // Advance start of full selection by one so the next character - // will already continue the string instead of having to type the - // same character again to start a new string. The selection - // includes formatting characters and is reverse when obtained from - // the Edit control. Selection aSel( GetSelection()); + + // Select the corresponding listbox entry if not current. This + // invalidates the Edit Selection thus has to happen between + // obtaining the Selection and setting the new Selection. + sal_Int32 nSelPos = ImplGetSelectEntryPos(); + if (nSelPos != nPos) + ImplSelectEntryPos( nPos, true); + + // If typing into the Edit control led us here, advance start of a + // full selection by one so the next character will already + // continue the string instead of having to type the same character + // again to start a new string. The selection includes formatting + // characters and is reverse when obtained from the Edit control. if (aSel.Max() == 1) { OUString aText( GetText()); if (aSel.Min() == aText.getLength()) - { ++aSel.Max(); - SetSelection( aSel); - } } + + SetSelection( aSel); + meEditedAndValid = EDITED_NO; } else |