commit 1b180c444d0c7d8b2df66c88f0bdadfec4896b00
author Caolán McNamara <caolanm@redhat.com> Mon Feb 13 20:17:51 2023 +0000
committer Caolán McNamara <caolanm@redhat.com> Tue Feb 14 11:33:57 2023 +0000
tree 5d907f3b743ee0eea2df9f1fb7cd8f49e6d4b77d
parent 20f3339834bedd4a62731f69c954dc6e6607bec5

Resolves: tdf#149482 don't insert if the mouse is not still pressed

In repeat callback, if we didn't see a mouse up, but find that the mouse
is no longer pressed at this point, then bail

Change-Id: I6e5d1d7201065fadcc2242fd52323ffb56766383
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146947
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

svtools/source/control/tabbar.cxx

diff --git a/svtools/source/control/tabbar.cxx b/svtools/source/control/tabbar.cxx
index b09c748..a0995ee 100644
--- a/svtools/source/control/tabbar.cxx
+++ b/svtools/source/control/tabbar.cxx
@@ -884,6 +884,15 @@ void TabBar::ImplShowPage( sal_uInt16 nPos )

IMPL_LINK( TabBar, ImplClickHdl, weld::Button&, rBtn, void )
{
    if ((GetPointerState().mnState & (MOUSE_LEFT | MOUSE_MIDDLE | MOUSE_RIGHT)) == 0)
    {
        // like tdf#149482 if we didn't see a mouse up, but find that the mouse is no
        // longer pressed at this point, then bail
        mpImpl->mxButtonBox->m_xPrevRepeater->Stop();
        mpImpl->mxButtonBox->m_xNextRepeater->Stop();
        return;
    }

    EndEditMode();

    sal_uInt16 nNewPos = mnFirstPos;
@@ -922,6 +931,14 @@ IMPL_LINK( TabBar, ImplClickHdl, weld::Button&, rBtn, void )

IMPL_LINK_NOARG(TabBar, ImplAddClickHandler, weld::Button&, void)
{
    if ((GetPointerState().mnState & (MOUSE_LEFT | MOUSE_MIDDLE | MOUSE_RIGHT)) == 0)
    {
        // tdf#149482 if we didn't see a mouse up, but find that the mouse is no
        // longer pressed at this point, then bail
        mpImpl->mxButtonBox->m_xAddRepeater->Stop();
        return;
    }

    EndEditMode();
    AddTabClick();
}