commit 899634104854ba5fedc616ff0e53b59ad61cc97a
author Caolán McNamara <caolanm@redhat.com> Thu May 27 20:37:50 2021 +0100
committer Caolán McNamara <caolanm@redhat.com> Fri May 28 11:22:09 2021 +0200
tree 090363e9b34cb0e4f8cde6b5e056a63b56e8eeaf
parent 676ba2babff95da3ac41205bc96d1bf252496d6f

tdf#142150 nothing already selected on MouseUp without previous MouseDown

if mouse down was on tooltip (possible on macOS)

Change-Id: Icb4328fa6977356c3bac5f37d42e99ae8d4e4b98
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116158
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
(cherry picked from commit 2a0d5643234cd0989f91589695694c809d82e344)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116159
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

svtools/source/control/valueset.cxx

diff --git a/svtools/source/control/valueset.cxx b/svtools/source/control/valueset.cxx
index 1d961ad..5ce7298 100644
--- a/svtools/source/control/valueset.cxx
+++ b/svtools/source/control/valueset.cxx
@@ -510,7 +510,9 @@ bool ValueSet::MouseButtonUp( const MouseEvent& rMouseEvent )
{
    if (rMouseEvent.IsLeft() && !rMouseEvent.IsMod2())
    {
        Select();
        // tdf#142150 MouseUp seen without previous MouseDown
        if (mnSelItemId)
            Select();
        return true;
    }