summaryrefslogtreecommitdiff
path: root/comphelper/source/misc/accessiblekeybindinghelper.cxx
diff options
context:
space:
mode:
authorMiklos Vajna <vmiklos@collabora.com>2022-04-26 15:40:44 +0200
committerMiklos Vajna <vmiklos@collabora.com>2022-04-26 17:02:15 +0200
commit651527b4efe9700c8c8dff58ce5aa86ad5681f16 (patch)
treedc963ba89c644d9a4f934ce628407e70bf9aca7c /comphelper/source/misc/accessiblekeybindinghelper.cxx
parent3524e4f7352993517ebdd514435378f6de0ca713 (diff)
sw: fix double-click opening frame dialog, not graphic dialog on images
The user-visible problem was that once a user clicks on an image a lot (e.g. 5 times), then the slot ID dispatched on double-click in SwEditWin::MouseButtonDown() is no longer FN_FORMAT_GRAFIC_DLG, but it's FN_FORMAT_FRAME_DLG. This is already inconsistent, but it's especially problematic in case an UNO client intercepts only the first UNO command, but not the second one. The other inconsistency is that in practice this only happens for as-char images, at-char anchored images work fine. The reason for this seems to be how we get the doc model position for a twips view point. At-char anchored images are handled at lcl_GetModelPositionForViewPoint_Objects(), and there we return the SwGrfNode in case the view point is inside the frame of the matching fly frame. SwTextCursor::GetModelPositionForViewPoint() restricted the same to as-char fly frames which have text or layout frame children. Fix the problem by allowing non-text frame children for as-char images. Change-Id: If08e7dd2a72f46ebcfb8c6ddf110703eaeb7df6d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133443 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Tested-by: Jenkins
Diffstat (limited to 'comphelper/source/misc/accessiblekeybindinghelper.cxx')
0 files changed, 0 insertions, 0 deletions