editXY: make cursor positionning correct over non editable inset

Instead of using complicated (and wrong) code, it is better to use
getPosNearX here.

Also simplify the code by removing temporary variables.

Fixes part of #10569.

diff --git a/src/TextMetrics.cpp b/src/TextMetrics.cpp
index 843ad7762ded41079ae0150ace17cfcb26425261..ddb38572ab11218067b43594ed47d9acd62c00c5 100644 (file)
--- a/src/TextMetrics.cpp
+++ b/src/TextMetrics.cpp
@@ -1330,23 +1330,19 @@ Inset * TextMetrics::editXY(Cursor & cur, int x, int y,
        }
        pit_type pit = getPitNearY(y);
        LASSERT(pit != -1, return 0);
-
-       int yy = y; // is modified by getPitAndRowNearY
-       Row const & row = getPitAndRowNearY(yy, pit, assert_in_view, up);
-
+       Row const & row = getPitAndRowNearY(y, pit, assert_in_view, up);
        cur.pit() = pit;
 
        // Do we cover an inset?
-       InsetList::InsetTable * it = checkInsetHit(pit, x, yy);
+       InsetList::InsetTable * it = checkInsetHit(pit, x, y);
 
        if (!it) {
                // No inset, set position in the text
                bool bound = false; // is modified by getPosNearX
-               int xx = x; // is modified by getPosNearX
-               cur.pos() = getPosNearX(row, xx, bound);
+               cur.pos() = getPosNearX(row, x, bound);
                cur.boundary(bound);
                cur.setCurrentFont();
-               cur.setTargetX(xx);
+               cur.setTargetX(x);
                return 0;
        }
 
@@ -1359,28 +1355,15 @@ Inset * TextMetrics::editXY(Cursor & cur, int x, int y,
        cur.setTargetX(x);
 
        // Try to descend recursively inside the inset.
-       Inset * edited = inset->editXY(cur, x, yy);
+       Inset * edited = inset->editXY(cur, x, y);
        if (edited == inset && cur.pos() == it->pos) {
                // non-editable inset, set cursor after the inset if x is
                // nearer to that position (bug 9628)
-               // TODO: This should be replaced with an improvement of
-               // Cursor::moveToClosestEdge that handles rtl text. (Which could not
-               // be tested because of #10569.)
-               CoordCache::Insets const & insetCache = bv_->coordCache().getInsets();
-               if (insetCache.has(inset)) {
-                       Dimension const & dim = insetCache.dim(inset);
-                       Point p = insetCache.xy(inset);
-                       bool const is_rtl = text_->isRTL(text_->getPar(pit));
-                       if (is_rtl) {
-                               // "in front of" == "right of"
-                               if (abs(p.x_ - x) < abs(p.x_ + dim.wid - x))
-                                       cur.posForward();
-                       } else {
-                               // "in front of" == "left of"
-                               if (abs(p.x_ + dim.wid - x) < abs(p.x_ - x))
-                                       cur.posForward();
-                       }
-               }
+               bool bound = false; // is modified by getPosNearX
+               cur.pos() = getPosNearX(row, x, bound);
+               cur.boundary(bound);
+               cur.setCurrentFont();
+               cur.setTargetX(x);
        }
 
        if (cur.top().text() == text_)