diff options
author | Armin Le Grand <Armin.Le.Grand@cib.de (CIB)> | 2018-03-28 16:55:19 +0200 |
---|---|---|
committer | Armin Le Grand <Armin.Le.Grand@cib.de> | 2018-03-29 12:00:23 +0200 |
commit | 7a6b66942260837e2a627b1df2a4fee9af1e3227 (patch) | |
tree | 0c9dd67ab9b08e20e7a613c65c6685027d26a346 /svx/source/svdraw | |
parent | 87b0eb0183e9a379a3e715dcb5de29b29e406686 (diff) |
Allow URL-Check for non-selected TableText
The part to detect a hit on a TableObjectText
in Draw/Impress for a non-selected Table was
setting the HitType to SdrHitKind::TextEditObj.
This prevents the latter happeing check for URL
(see below). Not directly setting to TextEditObj
is okay, this is determined and set later. I
checked various possible changes to keep the
behaviour as for 'normal' SdrObjects, so please
be careful when doing changes in the HitTest code
Change-Id: I326a73b88fc56fd902b5052cc3a2ddecf3c3ba61
Reviewed-on: https://gerrit.libreoffice.org/52016
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
Diffstat (limited to 'svx/source/svdraw')
-rw-r--r-- | svx/source/svdraw/svdview.cxx | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/svx/source/svdraw/svdview.cxx b/svx/source/svdraw/svdview.cxx index b5dbcd4959f8..1d44319efe5b 100644 --- a/svx/source/svdraw/svdview.cxx +++ b/svx/source/svdraw/svdview.cxx @@ -380,7 +380,12 @@ SdrHitKind SdrView::PickAnything(const Point& rLogicPos, SdrViewEvent& rVEvt) co eHit = SdrHitKind::Cell; break; case sdr::table::TableHitKind::CellTextArea: - eHit = SdrHitKind::TextEditObj; + // Keep state on UnmarkedObject to allow the below + // 'check for URL field' to be executed, else popups + // for e.g. URL links when hoovering and clicking + // them will not work. Tried several other changes, + // but this oje safely keeps existing behaviour as-is. + eHit = SdrHitKind::UnmarkedObject; break; default: break; |