diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2022-04-26 15:40:44 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2022-04-26 17:02:15 +0200 |
commit | 651527b4efe9700c8c8dff58ce5aa86ad5681f16 (patch) | |
tree | dc963ba89c644d9a4f934ce628407e70bf9aca7c /comphelper/source/misc/accessiblekeybindinghelper.cxx | |
parent | 3524e4f7352993517ebdd514435378f6de0ca713 (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