diff options
| author | Jim Raykowski <raykowj@gmail.com> | 2022-03-20 19:25:12 -0800 |
|---|---|---|
| committer | Jim Raykowski <raykowj@gmail.com> | 2022-03-24 17:54:10 +0100 |
| commit | d9f3a656c40e47d23a1c6de427bcdc7511d5249a (patch) | |
| tree | c132f82f1e2f983162f079462ce7c9b378c3b4d9 /vcl | |
| parent | ff3f11a7fa8b7f1096d0bb64a2aa41c4d1edae59 (diff) | |
fix gtk3 combobox up arrow and page up does not select first entry
when popup is not active
Change-Id: I0c4c0841bf0b24cc528d428829ec628e47032fb4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/131882
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Jim Raykowski <raykowj@gmail.com>
Diffstat (limited to 'vcl')
| -rw-r--r-- | vcl/unx/gtk3/gtkinst.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vcl/unx/gtk3/gtkinst.cxx b/vcl/unx/gtk3/gtkinst.cxx index 117c5c88a788..b738160118f2 100644 --- a/vcl/unx/gtk3/gtkinst.cxx +++ b/vcl/unx/gtk3/gtkinst.cxx @@ -20363,7 +20363,7 @@ private: sal_uInt16 nKeyMod = aKeyCode.GetModifier(); if (!nKeyMod) { - int nStartBound = m_bPopupActive ? 0 : (m_nMRUCount + 1); + int nStartBound = m_bPopupActive || !m_nMRUCount ? 0 : (m_nMRUCount + 1); int nActive = get_active_including_mru() - 1; while (nActive >= nStartBound && separator_function(nActive)) --nActive; @@ -20379,7 +20379,7 @@ private: if (!nKeyMod) { int nCount = get_count_including_mru(); - int nStartBound = m_bPopupActive ? 0 : (m_nMRUCount + 1); + int nStartBound = m_bPopupActive || !m_nMaxMRUCount ? 0 : (m_nMRUCount + 1); int nActive = nStartBound; while (nActive < nCount && separator_function(nActive)) ++nActive; |