From 04efc84abceb49e2042e357583a84b07847c1a5a Mon Sep 17 00:00:00 2001 From: Jean-Marc Lasgouttes Date: Thu, 8 Jun 2017 11:35:05 +0200 Subject: [PATCH] Fix bad cursor positioning when entering an inset The test added at 359aef92 was incorrect. Actually, Inset::editXY returns the inset when it is edited and also when it is not, which can be confusing. So if we are unlucky and the slices positions in the text area and in the inner inset match, the code below triggers and many bad things can happen. Also improve the documentation of Inset::editXY and TextMetrics::editXY. Fixes #10691. (cherry picked from commit c9fefd5a2cf5eb97fb23279e5b5e2f4c3a19b22e) --- src/TextMetrics.cpp | 4 +++- src/TextMetrics.h | 4 +++- src/insets/Inset.h | 8 +++++++- status.22x | 2 ++ 4 files changed, 15 insertions(+), 3 deletions(-) diff --git a/src/TextMetrics.cpp b/src/TextMetrics.cpp index 4f3e24d960..f61db67f72 100644 --- a/src/TextMetrics.cpp +++ b/src/TextMetrics.cpp @@ -1408,7 +1408,9 @@ Inset * TextMetrics::editXY(Cursor & cur, int x, int y, // Try to descend recursively inside the inset. Inset * edited = inset->editXY(cur, x, y); - if (edited == inset && cur.pos() == it->pos) { + // FIXME: it is not clear that the test on position is needed + // Remove it if/when semantics of editXY is clarified + if (cur.text() == text_ && cur.pos() == it->pos) { // non-editable inset, set cursor after the inset if x is // nearer to that position (bug 9628) bool bound = false; // is modified by getPosNearX diff --git a/src/TextMetrics.h b/src/TextMetrics.h index 6f5cf68733..016860be3d 100644 --- a/src/TextMetrics.h +++ b/src/TextMetrics.h @@ -180,7 +180,9 @@ public: that is completely visible \param up whether we are going up or down (only used when assert_in_view is true - \retval inset is non-null if the cursor is positionned inside + \retval inset is null if the cursor is positioned over normal + text in the current Text object. Otherwise it is the inset + that the cursor points to, like for Inset::editXY. */ /// FIXME: cleanup to use BufferView::getCoveringInset() and /// setCursorFromCoordinates() instead of checkInsetHit(). diff --git a/src/insets/Inset.h b/src/insets/Inset.h index cbcaf518a2..5eba4a4c8e 100644 --- a/src/insets/Inset.h +++ b/src/insets/Inset.h @@ -179,7 +179,13 @@ public: /// cursor enters virtual void edit(Cursor & cur, bool front, EntryDirection entry_from = ENTRY_DIRECTION_IGNORE); - /// cursor enters + /// sets cursor recursively descending into nested editable insets + /** + \return the inset pointer if x,y is covering that inset + \param x,y are absolute screen coordinates. + */ + /// Note: this method must preserve the selection status. See: + /// https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg199001.html virtual Inset * editXY(Cursor & cur, int x, int y); /// compute the size of the object returned in dim diff --git a/status.22x b/status.22x index 9bc48f3abe..8ecdefbf03 100644 --- a/status.22x +++ b/status.22x @@ -69,6 +69,8 @@ What's new - Fix glacial performance with ancient Hebrew text on macOS. +- Fix potential crash when cursor enters an inset (bug 10691). + * INTERNALS