svx/source/svdraw/svdhdl.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 46bbc6ff0f99f664ec4957cd29943b0627cbf2ba Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Thu Oct 12 18:57:57 2023 +0200 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Thu Nov 2 11:12:28 2023 +0100 tdf#141828 svx: Use pointing hand for custom shape handles Use the *pointing* hand cursor (`PointerStyle::RefHand`) instead of the hand symbol cursor (`PointerStyle::Hand`) for handles for custom shape interaction. (S. how `SdrHdlKind::CustomShape1` is only specified in `SdrObjCustomShape::AddToHdlList`, except for read-only switch/cases elsewhere). As mentioned in tdf#141828, this has the advantage that the user can see where the handle is being moved. For gtk3, this doesn't make any difference in practice, because it's using the same cursor type for both, `PointerStyle::RefHand` and `PointerStyle::Hand` (s. `GtkSalDisplay::getCursor`, but the Qt-based VCL plugins (`QtData::getCursor`) and macOS (s. `AquaSalFrame::getCurrentCursor`) make a distinction between them. Change-Id: I5d35481e5c5edc1895a7c4a973315ee6b187b983 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157891 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.wegh...@posteo.de> (cherry picked from commit 3d1cd490ac0366245c4b55d711611c21149a76e9) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158032 Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/svx/source/svdraw/svdhdl.cxx b/svx/source/svdraw/svdhdl.cxx index 379b3ed3f73e..64f29976f5ca 100644 --- a/svx/source/svdraw/svdhdl.cxx +++ b/svx/source/svdraw/svdhdl.cxx @@ -1011,7 +1011,7 @@ PointerStyle SdrHdl::GetPointer() const case SdrHdlKind::Ref2 : ePtr=PointerStyle::RefHand; break; case SdrHdlKind::BezierWeight : ePtr=PointerStyle::MoveBezierWeight; break; case SdrHdlKind::Glue : ePtr=PointerStyle::MovePoint; break; - case SdrHdlKind::CustomShape1 : ePtr=PointerStyle::Hand; break; + case SdrHdlKind::CustomShape1 : ePtr=PointerStyle::RefHand; break; default: break; }