]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
The bug-fix in revision 16531 introduced another bug. This is the right fix. I did...
[lyx.git] / src / text2.C
index c41a9c4c1eaae7f4e9f5ccae58cf7757fa526d7c..6b13d5b5d82956fad0523b068baee809f4e4e6ea 100644 (file)
@@ -748,147 +748,6 @@ void LyXText::setCurrentFont(LCursor & cur)
        }
 }
 
-
-// x is an absolute screen coord
-// returns the column near the specified x-coordinate of the row
-// x is set to the real beginning of this column
-pos_type LyXText::getColumnNearX(BufferView const & bv, int right_margin,
-               pit_type const pit, Row const & row, int & x, bool & boundary) const
-{
-       Buffer const & buffer = *bv.buffer();
-       TextMetrics const & tm = bv.textMetrics(this);
-       int const xo = bv.coordCache().get(this, pit).x_;
-       x -= xo;
-       RowMetrics const r = tm.computeRowMetrics(pit, row);
-       Paragraph const & par = pars_[pit];
-
-       pos_type vc = row.pos();
-       pos_type end = row.endpos();
-       pos_type c = 0;
-       LyXLayout_ptr const & layout = par.layout();
-
-       bool left_side = false;
-
-       pos_type body_pos = par.beginOfBody();
-
-       double tmpx = r.x;
-       double last_tmpx = tmpx;
-
-       if (body_pos > 0 &&
-           (body_pos > end || !par.isLineSeparator(body_pos - 1)))
-               body_pos = 0;
-
-       // check for empty row
-       if (vc == end) {
-               x = int(tmpx) + xo;
-               return 0;
-       }
-
-       frontend::FontMetrics const & fm 
-               = theFontMetrics(getLabelFont(buffer, par));
-
-       while (vc < end && tmpx <= x) {
-               c = bidi.vis2log(vc);
-               last_tmpx = tmpx;
-               if (body_pos > 0 && c == body_pos - 1) {
-                       // FIXME UNICODE
-                       docstring const lsep = from_utf8(layout->labelsep);
-                       tmpx += r.label_hfill + fm.width(lsep);
-                       if (par.isLineSeparator(body_pos - 1))
-                               tmpx -= singleWidth(buffer, par, body_pos - 1);
-               }
-
-               if (par.hfillExpansion(row, c)) {
-                       tmpx += singleWidth(buffer, par, c);
-                       if (c >= body_pos)
-                               tmpx += r.hfill;
-                       else
-                               tmpx += r.label_hfill;
-               } else if (par.isSeparator(c)) {
-                       tmpx += singleWidth(buffer, par, c);
-                       if (c >= body_pos)
-                               tmpx += r.separator;
-               } else {
-                       tmpx += singleWidth(buffer, par, c);
-               }
-               ++vc;
-       }
-
-       if ((tmpx + last_tmpx) / 2 > x) {
-               tmpx = last_tmpx;
-               left_side = true;
-       }
-
-       BOOST_ASSERT(vc <= end);  // This shouldn't happen.
-
-       boundary = false;
-       // This (rtl_support test) is not needed, but gives
-       // some speedup if rtl_support == false
-       bool const lastrow = lyxrc.rtl_support && row.endpos() == par.size();
-
-       // If lastrow is false, we don't need to compute
-       // the value of rtl.
-       bool const rtl = lastrow ? isRTL(buffer, par) : false;
-       if (lastrow &&
-           ((rtl  &&  left_side && vc == row.pos() && x < tmpx - 5) ||
-            (!rtl && !left_side && vc == end  && x > tmpx + 5)))
-               c = end;
-       else if (vc == row.pos()) {
-               c = bidi.vis2log(vc);
-               if (bidi.level(c) % 2 == 1)
-                       ++c;
-       } else {
-               c = bidi.vis2log(vc - 1);
-               bool const rtl = (bidi.level(c) % 2 == 1);
-               if (left_side == rtl) {
-                       ++c;
-                       boundary = bidi.isBoundary(buffer, par, c);
-               }
-       }
-
-// I believe this code is not needed anymore (Jug 20050717)
-#if 0
-       // The following code is necessary because the cursor position past
-       // the last char in a row is logically equivalent to that before
-       // the first char in the next row. That's why insets causing row
-       // divisions -- Newline and display-style insets -- must be treated
-       // specially, so cursor up/down doesn't get stuck in an air gap -- MV
-       // Newline inset, air gap below:
-       if (row.pos() < end && c >= end && par.isNewline(end - 1)) {
-               if (bidi.level(end -1) % 2 == 0)
-                       tmpx -= singleWidth(buffer, par, end - 1);
-               else
-                       tmpx += singleWidth(buffer, par, end - 1);
-               c = end - 1;
-       }
-
-       // Air gap above display inset:
-       if (row.pos() < end && c >= end && end < par.size()
-           && par.isInset(end) && par.getInset(end)->display()) {
-               c = end - 1;
-       }
-       // Air gap below display inset:
-       if (row.pos() < end && c >= end && par.isInset(end - 1)
-           && par.getInset(end - 1)->display()) {
-               c = end - 1;
-       }
-#endif
-
-       x = int(tmpx) + xo;
-       pos_type const col = c - row.pos();
-
-       if (!c || end == par.size())
-               return col;
-
-       if (c==end && !par.isLineSeparator(c-1) && !par.isNewline(c-1)) {
-               boundary = true;
-               return col;
-       }
-
-       return min(col, end - 1 - row.pos());
-}
-
-
 // y is screen coordinate
 pit_type LyXText::getPitNearY(BufferView & bv, int y)
 {
@@ -998,11 +857,9 @@ InsetBase * LyXText::editXY(LCursor & cur, int x, int y)
        bool bound = false;
 
        TextMetrics const & tm = cur.bv().textMetrics(this);
-       ParagraphMetrics const & pm = tm.parMetrics(pit);
-       int right_margin = tm.rightMargin(pm);
        int xx = x; // is modified by getColumnNearX
        pos_type const pos = row.pos()
-               + getColumnNearX(cur.bv(), right_margin, pit, row, xx, bound);
+               + tm.getColumnNearX(pit, row, xx, bound);
        cur.pit() = pit;
        cur.pos() = pos;
        cur.boundary(bound);
@@ -1124,7 +981,8 @@ bool LyXText::cursorUp(LCursor & cur)
        // Tell BufferView to test for FitCursor in any case!
        cur.updateFlags(Update::FitCursor);
 
-       ParagraphMetrics const & pm = cur.bv().parMetrics(this, cur.pit());
+       TextMetrics const & tm = cur.bv().textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(cur.pit());
 
        int row;
        int const x = cur.targetX();
@@ -1152,19 +1010,20 @@ bool LyXText::cursorUp(LCursor & cur)
                        ++dummy.pos();
 
                cur.bv().checkDepm(dummy, old);
+               return false;
        }
 
        bool updateNeeded = false;
 
        if (row > 0) {
                updateNeeded |= setCursor(cur, cur.pit(),
-                       x2pos(cur.bv(), cur.pit(), row - 1, x));
+                       tm.x2pos(cur.pit(), row - 1, x));
        } else if (cur.pit() > 0) {
                --cur.pit();
                //cannot use 'par' now
                ParagraphMetrics const & pmcur = cur.bv().parMetrics(this, cur.pit());
                updateNeeded |= setCursor(cur, cur.pit(),
-                       x2pos(cur.bv(), cur.pit(), pmcur.rows().size() - 1, x));
+                       tm.x2pos(cur.pit(), pmcur.rows().size() - 1, x));
        }
 
        cur.x_target() = x;
@@ -1178,7 +1037,8 @@ bool LyXText::cursorDown(LCursor & cur)
        // Tell BufferView to test for FitCursor in any case!
        cur.updateFlags(Update::FitCursor);
 
-       ParagraphMetrics const & pm = cur.bv().parMetrics(this, cur.pit());
+       TextMetrics const & tm = cur.bv().textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(cur.pit());
 
        int row;
        int const x = cur.targetX();
@@ -1217,11 +1077,11 @@ bool LyXText::cursorDown(LCursor & cur)
 
        if (row + 1 < int(pm.rows().size())) {
                updateNeeded |= setCursor(cur, cur.pit(),
-                       x2pos(cur.bv(), cur.pit(), row + 1, x));
+                       tm.x2pos(cur.pit(), row + 1, x));
        } else if (cur.pit() + 1 < int(paragraphs().size())) {
                ++cur.pit();
                updateNeeded |= setCursor(cur, cur.pit(),
-                       x2pos(cur.bv(), cur.pit(), 0, x));
+                       tm.x2pos(cur.pit(), 0, x));
        }
 
        cur.x_target() = x;
@@ -1276,8 +1136,6 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur,
                LCursor & old, bool & need_anchor_change)
 {
        //lyxerr[Debug::DEBUG] << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
-       // old should point to us
-       BOOST_ASSERT(old.text() == this);
 
        Paragraph & oldpar = old.paragraph();