diff options
author | Attila Szűcs <szucs.attila3@nisz.hu> | 2020-10-16 09:54:14 +0200 |
---|---|---|
committer | Gabor Kelemen <kelemen.gabor2@nisz.hu> | 2021-04-20 14:59:37 +0200 |
commit | 84757610daa040eb3aa7aa5b5902697ee2cf83e1 (patch) | |
tree | 948c224e8dbc1d27b8d82b590dc88ea0513e8628 | |
parent | e3bf53d92ee9c933e7d5794989facb24abf4315d (diff) |
tdf#116675 vcl tree list: toggle by label click (e.g. in AutoFilter)
Checkboxes in a tree list could be toggled only by clicking
on them, but not on the associated labels, despite UX guidelines
and interoperability requirements especially for AutoFilter usage
in the spreadsheet.
Note: Implemented only for generic VCL plugin. Testing on Linux:
SAL_USE_VCLPLUGIN=gen instdir/program/soffice
Co-authored-by: Tibor Nagy (NISZ)
Change-Id: Ibc3d42271a1be53a54e11dedf4ab3ab506d680cf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104414
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
(cherry picked from commit 2471d6f44c7e8ecbe86a90eeb593b899a08a7408)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114287
Tested-by: Gabor Kelemen <kelemen.gabor2@nisz.hu>
Reviewed-by: Gabor Kelemen <kelemen.gabor2@nisz.hu>
-rw-r--r-- | vcl/source/treelist/treelistbox.cxx | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/vcl/source/treelist/treelistbox.cxx b/vcl/source/treelist/treelistbox.cxx index bb52e28fe0d4..bbece8b674d9 100644 --- a/vcl/source/treelist/treelistbox.cxx +++ b/vcl/source/treelist/treelistbox.cxx @@ -2329,6 +2329,29 @@ void SvTreeListBox::MouseButtonDown( const MouseEvent& rMEvt ) void SvTreeListBox::MouseButtonUp( const MouseEvent& rMEvt ) { + // tdf#116675 clicking on an entry should toggle its checkbox + if (rMEvt.IsLeft() && (nTreeFlags & SvTreeFlags::CHKBTN)) + { + const Point aPnt = rMEvt.GetPosPixel(); + SvTreeListEntry* pEntry = GetEntry(aPnt); + if (pEntry && pEntry->m_Items.size() > 0) + { + SvLBoxItem* pItem = GetItem(pEntry, aPnt.X()); + // if the checkbox button was clicked, that will be toggled later, do not toggle here + // anyway users probably don't want to toggle the checkbox by clickink on another button + if (!pItem || pItem->GetType() != SvLBoxItemType::Button) + { + SvLBoxButton* pItemCheckBox + = static_cast<SvLBoxButton*>(pEntry->GetFirstItem(SvLBoxItemType::Button)); + if (pItemCheckBox) + { + pItemCheckBox->ClickHdl(pEntry); + InvalidateEntry(pEntry); + } + } + } + } + pImpl->MouseButtonUp( rMEvt ); } |