]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
Clean-up FileFilterList API.
[lyx.git] / src / text2.C
index 426cb90be03222c7d00131b5bbe8c411d4429b8f..8dad53a82a89c9f0c5312ea89a15b60796beccbf 100644 (file)
@@ -27,6 +27,7 @@
 #include "BufferView.h"
 #include "Bullet.h"
 #include "counters.h"
+#include "coordcache.h"
 #include "cursor.h"
 #include "CutAndPaste.h"
 #include "debug.h"
 #include "support/lstrings.h"
 #include "support/textutils.h"
 #include "support/tostr.h"
-#include "support/std_sstream.h"
 
-using lyx::par_type;
+#include <sstream>
+
+using lyx::pit_type;
 using lyx::pos_type;
 using lyx::support::bformat;
 
@@ -84,11 +86,11 @@ void LyXText::init(BufferView * bv)
        width_ = maxwidth_;
        height_ = 0;
 
-       par_type const end = paragraphs().size();
-       for (par_type pit = 0; pit != end; ++pit)
+       pit_type const end = paragraphs().size();
+       for (pit_type pit = 0; pit != end; ++pit)
                pars_[pit].rows.clear();
 
-       current_font = getFont(0, 0);
+       current_font = getFont(pars_[0], 0);
        redoParagraphs(0, end);
        updateCounters();
 }
@@ -100,25 +102,67 @@ bool LyXText::isMainText() const
 }
 
 
+// takes absolute x,y coordinates
+InsetBase * LyXText::checkInsetHit(int x, int y) const
+{
+       pit_type pit;
+       pit_type end;
+
+       getParsInRange(paragraphs(),
+                      bv()->top_y() - yo_,
+                      bv()->top_y() - yo_ + bv()->workHeight(),
+                      pit, end);
+
+       // convert to screen-absolute y coordinate
+       y -= bv()->top_y();
+       lyxerr << "checkInsetHit: x: " << x << " y: " << y << endl;
+       lyxerr << "  pit: " << pit << " end: " << end << endl;
+       for (; pit != end; ++pit) {
+               InsetList::const_iterator iit = pars_[pit].insetlist.begin();
+               InsetList::const_iterator iend = pars_[pit].insetlist.end();
+               for (; iit != iend; ++iit) {
+                       InsetBase * inset = iit->inset;
+#if 1
+                       lyxerr << "examining inset " << inset << endl;
+                       if (theCoords.insets_.has(inset))
+                               lyxerr
+                                       << " xo: " << inset->xo() << "..." << inset->xo() + inset->width()
+                                       << " yo: " << inset->yo() - inset->ascent() << "..."
+                                       << inset->yo() + inset->descent() << endl;
+                       else
+                               lyxerr << " inset has no cached position";
+#endif
+                       if (inset->covers(x, y)) {
+                               lyxerr << "Hit inset: " << inset << endl;
+                               return inset;
+                       }
+               }
+       }
+       lyxerr << "No inset hit. " << endl;
+       return 0;
+}
+
+
+
 // Gets the fully instantiated font at a given position in a paragraph
 // Basically the same routine as Paragraph::getFont() in paragraph.C.
 // The difference is that this one is used for displaying, and thus we
 // are allowed to make cosmetic improvements. For instance make footnotes
 // smaller. (Asger)
-LyXFont LyXText::getFont(par_type pit, pos_type pos) const
+LyXFont LyXText::getFont(Paragraph const & par, pos_type const pos) const
 {
        BOOST_ASSERT(pos >= 0);
 
-       LyXLayout_ptr const & layout = pars_[pit].layout();
+       LyXLayout_ptr const & layout = par.layout();
 #ifdef WITH_WARNINGS
 #warning broken?
 #endif
        BufferParams const & params = bv()->buffer()->params();
-       pos_type const body_pos = pars_[pit].beginOfBody();
+       pos_type const body_pos = par.beginOfBody();
 
        // We specialize the 95% common case:
-       if (!pars_[pit].getDepth()) {
-               LyXFont f = pars_[pit].getFontSettings(params, pos);
+       if (!par.getDepth()) {
+               LyXFont f = par.getFontSettings(params, pos);
                if (!isMainText())
                        f.realize(font_);
                if (layout->labeltype == LABEL_MANUAL && pos < body_pos)
@@ -134,21 +178,20 @@ LyXFont LyXText::getFont(par_type pit, pos_type pos) const
        else
                layoutfont = layout->font;
 
-       LyXFont font = pars_[pit].getFontSettings(params, pos);
+       LyXFont font = par.getFontSettings(params, pos);
        font.realize(layoutfont);
 
        if (!isMainText())
                font.realize(font_);
 
        // Realize with the fonts of lesser depth.
-       //font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
 }
 
 
-LyXFont LyXText::getLayoutFont(par_type pit) const
+LyXFont LyXText::getLayoutFont(pit_type const pit) const
 {
        LyXLayout_ptr const & layout = pars_[pit].layout();
 
@@ -164,23 +207,22 @@ LyXFont LyXText::getLayoutFont(par_type pit) const
 }
 
 
-LyXFont LyXText::getLabelFont(par_type pit) const
+LyXFont LyXText::getLabelFont(Paragraph const & par) const
 {
-       LyXLayout_ptr const & layout = pars_[pit].layout();
+       LyXLayout_ptr const & layout = par.layout();
 
-       if (!pars_[pit].getDepth())
+       if (!par.getDepth())
                return layout->reslabelfont;
 
        LyXFont font = layout->labelfont;
        // Realize with the fonts of lesser depth.
-       font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
 }
 
 
-void LyXText::setCharFont(par_type pit, pos_type pos, LyXFont const & fnt)
+void LyXText::setCharFont(pit_type pit, pos_type pos, LyXFont const & fnt)
 {
        LyXFont font = fnt;
        LyXLayout_ptr const & layout = pars_[pit].layout();
@@ -195,12 +237,12 @@ void LyXText::setCharFont(par_type pit, pos_type pos, LyXFont const & fnt)
 
        // Realize against environment font information
        if (pars_[pit].getDepth()) {
-               par_type tp = pit;
+               pit_type tp = pit;
                while (!layoutfont.resolved() &&
-                      tp != par_type(paragraphs().size()) &&
+                      tp != pit_type(paragraphs().size()) &&
                       pars_[tp].getDepth()) {
                        tp = outerHook(tp, paragraphs());
-                       if (tp != par_type(paragraphs().size()))
+                       if (tp != pit_type(paragraphs().size()))
                                layoutfont.realize(pars_[tp].layout()->font);
                }
        }
@@ -237,10 +279,10 @@ void LyXText::makeFontEntriesLayoutSpecific(BufferParams const & params,
 
 
 // return past-the-last paragraph influenced by a layout change on pit
-par_type LyXText::undoSpan(par_type pit)
+pit_type LyXText::undoSpan(pit_type pit)
 {
-       par_type end = paragraphs().size();
-       par_type nextpit = pit + 1;
+       pit_type end = paragraphs().size();
+       pit_type nextpit = pit + 1;
        if (nextpit == end)
                return nextpit;
        //because of parindents
@@ -255,16 +297,16 @@ par_type LyXText::undoSpan(par_type pit)
 }
 
 
-par_type LyXText::setLayout(par_type start, par_type end, string const & layout)
+pit_type LyXText::setLayout(pit_type start, pit_type end, string const & layout)
 {
        BOOST_ASSERT(start != end);
-       par_type undopit = undoSpan(end - 1);
+       pit_type undopit = undoSpan(end - 1);
        recUndo(start, undopit - 1);
 
        BufferParams const & bufparams = bv()->buffer()->params();
        LyXLayout_ptr const & lyxlayout = bufparams.getLyXTextClass()[layout];
 
-       for (par_type pit = start; pit != end; ++pit) {
+       for (pit_type pit = start; pit != end; ++pit) {
                pars_[pit].applyLayout(lyxlayout);
                makeFontEntriesLayoutSpecific(bufparams, pars_[pit]);
                if (lyxlayout->margintype == MARGIN_MANUAL)
@@ -285,7 +327,7 @@ void LyXText::setLayout(LCursor & cur, string const & layout)
        LyXLayout_ptr const & lyxlayout = params.getLyXTextClass()[layout];
        if (lyxlayout->is_environment) {
                // move everything in a new environment inset
-               lyxerr << "setting layout " << layout << endl;
+               lyxerr[Debug::DEBUG] << "setting layout " << layout << endl;
                bv.owner()->dispatch(FuncRequest(LFUN_HOME));
                bv.owner()->dispatch(FuncRequest(LFUN_ENDSEL));
                bv.owner()->dispatch(FuncRequest(LFUN_CUT));
@@ -296,9 +338,9 @@ void LyXText::setLayout(LCursor & cur, string const & layout)
                return;
        }
 
-       par_type start = cur.selBegin().par();
-       par_type end = cur.selEnd().par() + 1;
-       par_type endpit = setLayout(start, end, layout);
+       pit_type start = cur.selBegin().pit();
+       pit_type end = cur.selEnd().pit() + 1;
+       pit_type endpit = setLayout(start, end, layout);
        redoParagraphs(start, endpit);
        updateCounters();
 }
@@ -307,14 +349,14 @@ void LyXText::setLayout(LCursor & cur, string const & layout)
 namespace {
 
 
-void getSelectionSpan(LCursor & cur, par_type & beg, par_type & end)
+void getSelectionSpan(LCursor & cur, pit_type & beg, pit_type & end)
 {
        if (!cur.selection()) {
-               beg = cur.par();
-               end = cur.par() + 1;
+               beg = cur.pit();
+               end = cur.pit() + 1;
        } else {
-               beg = cur.selBegin().par();
-               end = cur.selEnd().par() + 1;
+               beg = cur.selBegin().pit();
+               end = cur.selEnd().pit() + 1;
        }
 }
 
@@ -339,13 +381,13 @@ bool changeDepthAllowed(LyXText::DEPTH_CHANGE type,
 bool LyXText::changeDepthAllowed(LCursor & cur, DEPTH_CHANGE type) const
 {
        BOOST_ASSERT(this == cur.text());
-       par_type beg, end;
+       pit_type beg, end;
        getSelectionSpan(cur, beg, end);
        int max_depth = 0;
        if (beg != 0)
                max_depth = pars_[beg - 1].getMaxDepthAfter();
 
-       for (par_type pit = beg; pit != end; ++pit) {
+       for (pit_type pit = beg; pit != end; ++pit) {
                if (::changeDepthAllowed(type, pars_[pit], max_depth))
                        return true;
                max_depth = pars_[pit].getMaxDepthAfter();
@@ -357,7 +399,7 @@ bool LyXText::changeDepthAllowed(LCursor & cur, DEPTH_CHANGE type) const
 void LyXText::changeDepth(LCursor & cur, DEPTH_CHANGE type)
 {
        BOOST_ASSERT(this == cur.text());
-       par_type beg, end;
+       pit_type beg, end;
        getSelectionSpan(cur, beg, end);
        recordUndoSelection(cur);
 
@@ -365,7 +407,7 @@ void LyXText::changeDepth(LCursor & cur, DEPTH_CHANGE type)
        if (beg != 0)
                max_depth = pars_[beg - 1].getMaxDepthAfter();
 
-       for (par_type pit = beg; pit != end; ++pit) {
+       for (pit_type pit = beg; pit != end; ++pit) {
                if (::changeDepthAllowed(type, pars_[pit], max_depth)) {
                        int const depth = pars_[pit].params().depth();
                        if (type == INC_DEPTH)
@@ -389,9 +431,9 @@ void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
        if (!cur.selection()) {
                // Determine basis font
                LyXFont layoutfont;
-               par_type pit = cur.par();
+               pit_type pit = cur.pit();
                if (cur.pos() < pars_[pit].beginOfBody())
-                       layoutfont = getLabelFont(pit);
+                       layoutfont = getLabelFont(pars_[pit]);
                else
                        layoutfont = getLayoutFont(pit);
 
@@ -412,23 +454,21 @@ void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
        // Ok, we have a selection.
        recordUndoSelection(cur);
 
-       par_type const beg = cur.selBegin().par();
-       par_type const end = cur.selEnd().par();
-
-       DocIterator pos = cur.selectionBegin();
-       DocIterator posend = cur.selectionEnd();
+       pit_type const beg = cur.selBegin().pit();
+       pit_type const end = cur.selEnd().pit();
 
-       lyxerr << "pos: " << pos << " posend: " << posend << endl;
+       DocIterator dit = cur.selectionBegin();
+       DocIterator ditend = cur.selectionEnd();
 
        BufferParams const & params = cur.buffer().params();
 
-       // Don't use forwardChar here as posend might have
+       // Don't use forwardChar here as ditend might have
        // pos() == lastpos() and forwardChar would miss it.
-       for (; pos != posend; pos.forwardPos()) {
-               if (pos.pos() != pos.lastpos()) {
-                       LyXFont f = getFont(pos.par(), pos.pos());
+       for (; dit != ditend; dit.forwardPos()) {
+               if (dit.pos() != dit.lastpos()) {
+                       LyXFont f = getFont(dit.paragraph(), dit.pos());
                        f.update(font, params.language, toggleall);
-                       setCharFont(pos.par(), pos.pos(), f);
+                       setCharFont(dit.pit(), dit.pos(), f);
                }
        }
 
@@ -442,7 +482,7 @@ void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
 void LyXText::cursorHome(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       setCursor(cur, cur.par(), cur.textRow().pos());
+       setCursor(cur, cur.pit(), cur.textRow().pos());
 }
 
 
@@ -452,7 +492,7 @@ void LyXText::cursorEnd(LCursor & cur)
        // if not on the last row of the par, put the cursor before
        // the final space
        pos_type const end = cur.textRow().endpos();
-       setCursor(cur, cur.par(), end == cur.lastpos() ? end : end - 1);
+       setCursor(cur, cur.pit(), end == cur.lastpos() ? end : end - 1);
 }
 
 
@@ -466,7 +506,7 @@ void LyXText::cursorTop(LCursor & cur)
 void LyXText::cursorBottom(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       setCursor(cur, cur.lastpar(), boost::prior(paragraphs().end())->size());
+       setCursor(cur, cur.lastpit(), boost::prior(paragraphs().end())->size());
 }
 
 
@@ -516,7 +556,7 @@ string LyXText::getStringToIndex(LCursor & cur)
        string idxstring;
        if (!cur.selection())
                cur.message(_("Nothing to index!"));
-       else if (cur.selBegin().par() != cur.selEnd().par())
+       else if (cur.selBegin().pit() != cur.selEnd().pit())
                cur.message(_("Cannot index more than one paragraph!"));
        else
                idxstring = cur.selectionAsString(false);
@@ -539,10 +579,10 @@ void LyXText::setParagraph(LCursor & cur,
 {
        BOOST_ASSERT(cur.text());
        // make sure that the depth behind the selection are restored, too
-       par_type undopit = undoSpan(cur.selEnd().par());
-       recUndo(cur.selBegin().par(), undopit - 1);
+       pit_type undopit = undoSpan(cur.selEnd().pit());
+       recUndo(cur.selBegin().pit(), undopit - 1);
 
-       for (par_type pit = cur.selBegin().par(), end = cur.selEnd().par();
+       for (pit_type pit = cur.selBegin().pit(), end = cur.selEnd().pit();
                        pit <= end; ++pit) {
                Paragraph & par = pars_[pit];
                ParagraphParameters & params = par.params();
@@ -563,7 +603,7 @@ void LyXText::setParagraph(LCursor & cur,
                params.noindent(noindent);
        }
 
-       redoParagraphs(cur.selBegin().par(), undopit);
+       redoParagraphs(cur.selBegin().pit(), undopit);
 }
 
 
@@ -591,7 +631,7 @@ string expandLabel(LyXTextClass const & textclass,
 
 namespace {
 
-void incrementItemDepth(ParagraphList & pars, par_type pit, par_type first_pit)
+void incrementItemDepth(ParagraphList & pars, pit_type pit, pit_type first_pit)
 {
        int const cur_labeltype = pars[pit].layout()->labeltype;
 
@@ -600,7 +640,7 @@ void incrementItemDepth(ParagraphList & pars, par_type pit, par_type first_pit)
 
        int const cur_depth = pars[pit].getDepth();
 
-       par_type prev_pit = pit - 1;
+       pit_type prev_pit = pit - 1;
        while (true) {
                int const prev_depth = pars[prev_pit].getDepth();
                int const prev_labeltype = pars[prev_pit].layout()->labeltype;
@@ -628,14 +668,14 @@ void incrementItemDepth(ParagraphList & pars, par_type pit, par_type first_pit)
 }
 
 
-void resetEnumCounterIfNeeded(ParagraphList & pars, par_type pit,
-       par_type firstpit, Counters & counters)
+void resetEnumCounterIfNeeded(ParagraphList & pars, pit_type pit,
+       pit_type firstpit, Counters & counters)
 {
        if (pit == firstpit)
                return;
 
        int const cur_depth = pars[pit].getDepth();
-       par_type prev_pit = pit - 1;
+       pit_type prev_pit = pit - 1;
        while (true) {
                int const prev_depth = pars[prev_pit].getDepth();
                int const prev_labeltype = pars[prev_pit].layout()->labeltype;
@@ -666,39 +706,39 @@ void resetEnumCounterIfNeeded(ParagraphList & pars, par_type pit,
 
 
 // set the counter of a paragraph. This includes the labels
-void LyXText::setCounter(Buffer const & buf, par_type pit)
+void LyXText::setCounter(Buffer const & buf, pit_type pit)
 {
+       Paragraph & par = pars_[pit];
        BufferParams const & bufparams = buf.params();
        LyXTextClass const & textclass = bufparams.getLyXTextClass();
-       LyXLayout_ptr const & layout = pars_[pit].layout();
-       par_type first_pit = 0;
+       LyXLayout_ptr const & layout = par.layout();
        Counters & counters = textclass.counters();
 
        // Always reset
-       pars_[pit].itemdepth = 0;
+       par.itemdepth = 0;
 
-       if (pit == first_pit) {
-               pars_[pit].params().appendix(pars_[pit].params().startOfAppendix());
+       if (pit == 0) {
+               par.params().appendix(par.params().startOfAppendix());
        } else {
-               pars_[pit].params().appendix(pars_[pit - 1].params().appendix());
-               if (!pars_[pit].params().appendix() &&
-                   pars_[pit].params().startOfAppendix()) {
-                       pars_[pit].params().appendix(true);
+               par.params().appendix(pars_[pit - 1].params().appendix());
+               if (!par.params().appendix() &&
+                   par.params().startOfAppendix()) {
+                       par.params().appendix(true);
                        textclass.counters().reset();
                }
 
                // Maybe we have to increment the item depth.
-               incrementItemDepth(pars_, pit, first_pit);
+               incrementItemDepth(pars_, pit, 0);
        }
 
        // erase what was there before
-       pars_[pit].params().labelString(string());
+       par.params().labelString(string());
 
        if (layout->margintype == MARGIN_MANUAL) {
-               if (pars_[pit].params().labelWidthString().empty())
-                       pars_[pit].setLabelWidthString(layout->labelstring());
+               if (par.params().labelWidthString().empty())
+                       par.setLabelWidthString(layout->labelstring());
        } else {
-               pars_[pit].setLabelWidthString(string());
+               par.setLabelWidthString(string());
        }
 
        // is it a layout that has an automatic label?
@@ -706,16 +746,16 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
                BufferParams const & bufparams = buf.params();
                LyXTextClass const & textclass = bufparams.getLyXTextClass();
                counters.step(layout->counter);
-               string label = expandLabel(textclass, layout, pars_[pit].params().appendix());
-               pars_[pit].params().labelString(label);
+               string label = expandLabel(textclass, layout, par.params().appendix());
+               par.params().labelString(label);
        } else if (layout->labeltype == LABEL_ITEMIZE) {
                // At some point of time we should do something more
                // clever here, like:
-               //   pars_[pit].params().labelString(
-               //    bufparams.user_defined_bullet(pars_[pit].itemdepth).getText());
+               //   par.params().labelString(
+               //    bufparams.user_defined_bullet(par.itemdepth).getText());
                // for now, use a simple hardcoded label
                string itemlabel;
-               switch (pars_[pit].itemdepth) {
+               switch (par.itemdepth) {
                case 0:
                        itemlabel = "*";
                        break;
@@ -730,17 +770,17 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
                        break;
                }
 
-               pars_[pit].params().labelString(itemlabel);
+               par.params().labelString(itemlabel);
        } else if (layout->labeltype == LABEL_ENUMERATE) {
                // Maybe we have to reset the enumeration counter.
-               resetEnumCounterIfNeeded(pars_, pit, first_pit, counters);
+               resetEnumCounterIfNeeded(pars_, pit, 0, counters);
 
                // FIXME
                // Yes I know this is a really, really! bad solution
                // (Lgb)
                string enumcounter = "enum";
 
-               switch (pars_[pit].itemdepth) {
+               switch (par.itemdepth) {
                case 2:
                        enumcounter += 'i';
                case 1:
@@ -758,13 +798,13 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
 
                counters.step(enumcounter);
 
-               pars_[pit].params().labelString(counters.enumLabel(enumcounter));
+               par.params().labelString(counters.enumLabel(enumcounter));
        } else if (layout->labeltype == LABEL_BIBLIO) {// ale970302
                counters.step("bibitem");
                int number = counters.value("bibitem");
-               if (pars_[pit].bibitem()) {
-                       pars_[pit].bibitem()->setCounter(number);
-                       pars_[pit].params().labelString(layout->labelstring());
+               if (par.bibitem()) {
+                       par.bibitem()->setCounter(number);
+                       par.params().labelString(layout->labelstring());
                }
                // In biblio should't be following counters but...
        } else {
@@ -772,8 +812,8 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
 
                // the caption hack:
                if (layout->labeltype == LABEL_SENSITIVE) {
-                       par_type end = paragraphs().size();
-                       par_type tmppit = pit;
+                       pit_type end = paragraphs().size();
+                       pit_type tmppit = pit;
                        InsetBase * in = 0;
                        bool isOK = false;
                        while (tmppit != end) {
@@ -782,13 +822,27 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
                                    in->lyxCode() == InsetBase::WRAP_CODE) {
                                        isOK = true;
                                        break;
-                               } else {
+                               }
+#ifdef WITH_WARNINGS
+#warning replace this code by something that works
+// This code does not work because we have currently no way to move up
+// in the hierarchy of insets (JMarc 16/08/2004)
+#endif
+#if 0
+/* I think this code is supposed to be useful when one has a caption
+ * in a minipage in a figure inset. We need to go up to be able to see
+ * that the caption should use "Figure" as label
+ */
+                               else {
                                        Paragraph const * owner = &ownerPar(buf, in);
-                                       tmppit = first_pit;
+                                       tmppit = 0;
                                        for ( ; tmppit != end; ++tmppit)
                                                if (&pars_[tmppit] == owner)
                                                        break;
                                }
+#else
+                               ++tmppit;
+#endif
                        }
 
                        if (isOK) {
@@ -813,7 +867,7 @@ void LyXText::setCounter(Buffer const & buf, par_type pit)
                                s = _("Senseless: ");
                        }
                }
-               pars_[pit].params().labelString(s);
+               par.params().labelString(s);
 
        }
 }
@@ -827,8 +881,8 @@ void LyXText::updateCounters()
 
        bool update_pos = false;
 
-       par_type end = paragraphs().size();
-       for (par_type pit = 0; pit != end; ++pit) {
+       pit_type end = paragraphs().size();
+       for (pit_type pit = 0; pit != end; ++pit) {
                string const oldLabel = pars_[pit].params().labelString();
                size_t maxdepth = 0;
                if (pit != 0)
@@ -841,7 +895,7 @@ void LyXText::updateCounters()
                setCounter(*bv()->buffer(), pit);
                string const & newLabel = pars_[pit].params().labelString();
                if (oldLabel != newLabel) {
-                       //lyxerr << "changing labels: old: " << oldLabel << " new: "
+                       //lyxerr[Debug::DEBUG] << "changing labels: old: " << oldLabel << " new: "
                        //      << newLabel << endl;
                        redoParagraphInternal(pit);
                        update_pos = true;
@@ -852,6 +906,7 @@ void LyXText::updateCounters()
 }
 
 
+// this really should just insert the inset and not move the cursor.
 void LyXText::insertInset(LCursor & cur, InsetBase * inset)
 {
        BOOST_ASSERT(this == cur.text());
@@ -864,8 +919,8 @@ void LyXText::insertInset(LCursor & cur, InsetBase * inset)
 // needed to insert the selection
 void LyXText::insertStringAsLines(LCursor & cur, string const & str)
 {
-       par_type pit = cur.par();
-       par_type endpit = cur.par() + 1;
+       pit_type pit = cur.pit();
+       pit_type endpit = cur.pit() + 1;
        pos_type pos = cur.pos();
        recordUndo(cur);
 
@@ -873,9 +928,9 @@ void LyXText::insertStringAsLines(LCursor & cur, string const & str)
        cur.clearSelection();
        cur.buffer().insertStringAsLines(pars_, pit, pos, current_font, str);
 
-       redoParagraphs(cur.par(), endpit);
+       redoParagraphs(cur.pit(), endpit);
        cur.resetAnchor();
-       setCursor(cur, cur.par(), pos);
+       setCursor(cur, cur.pit(), pos);
        cur.setSelection();
 }
 
@@ -907,7 +962,7 @@ void LyXText::insertStringAsParagraphs(LCursor & cur, string const & str)
 }
 
 
-bool LyXText::setCursor(LCursor & cur, par_type par, pos_type pos,
+bool LyXText::setCursor(LCursor & cur, pit_type par, pos_type pos,
        bool setfont, bool boundary)
 {
        LCursor old = cur;
@@ -916,12 +971,12 @@ bool LyXText::setCursor(LCursor & cur, par_type par, pos_type pos,
 }
 
 
-void LyXText::setCursor(CursorSlice & cur, par_type par,
+void LyXText::setCursor(CursorSlice & cur, pit_type par,
        pos_type pos, bool boundary)
 {
        BOOST_ASSERT(par != int(paragraphs().size()));
 
-       cur.par() = par;
+       cur.pit() = par;
        cur.pos() = pos;
        cur.boundary() = boundary;
 
@@ -968,7 +1023,7 @@ void LyXText::setCursor(CursorSlice & cur, par_type par,
 
 
 void LyXText::setCursorIntern(LCursor & cur,
-       par_type par, pos_type pos, bool setfont, bool boundary)
+       pit_type par, pos_type pos, bool setfont, bool boundary)
 {
        setCursor(cur.top(), par, pos, boundary);
        cur.x_target() = cursorX(cur.top());
@@ -981,7 +1036,7 @@ void LyXText::setCurrentFont(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
        pos_type pos = cur.pos();
-       par_type pit = cur.par();
+       Paragraph & par = cur.paragraph();
 
        if (cur.boundary() && pos > 0)
                --pos;
@@ -990,7 +1045,7 @@ void LyXText::setCurrentFont(LCursor & cur)
                if (pos == cur.lastpos())
                        --pos;
                else // potentional bug... BUG (Lgb)
-                       if (pars_[pit].isSeparator(pos)) {
+                       if (par.isSeparator(pos)) {
                                if (pos > cur.textRow().pos() &&
                                    bidi.level(pos) % 2 ==
                                    bidi.level(pos - 1) % 2)
@@ -1001,13 +1056,13 @@ void LyXText::setCurrentFont(LCursor & cur)
        }
 
        BufferParams const & bufparams = cur.buffer().params();
-       current_font = pars_[pit].getFontSettings(bufparams, pos);
-       real_current_font = getFont(pit, pos);
+       current_font = par.getFontSettings(bufparams, pos);
+       real_current_font = getFont(par, pos);
 
        if (cur.pos() == cur.lastpos()
-           && bidi.isBoundary(cur.buffer(), pars_[pit], cur.pos())
+           && bidi.isBoundary(cur.buffer(), par, cur.pos())
            && !cur.boundary()) {
-               Language const * lang = pars_[pit].getParLanguage(bufparams);
+               Language const * lang = par.getParLanguage(bufparams);
                current_font.setLanguage(lang);
                current_font.setNumber(LyXFont::OFF);
                real_current_font.setLanguage(lang);
@@ -1019,26 +1074,27 @@ 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(par_type pit,
+pos_type LyXText::getColumnNearX(pit_type const pit,
        Row const & row, int & x, bool & boundary) const
 {
        x -= xo_;
        RowMetrics const r = 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 = pars_[pit].layout();
+       LyXLayout_ptr const & layout = par.layout();
 
        bool left_side = false;
 
-       pos_type body_pos = pars_[pit].beginOfBody();
+       pos_type body_pos = par.beginOfBody();
 
        double tmpx = r.x;
        double last_tmpx = tmpx;
 
        if (body_pos > 0 &&
-           (body_pos > end || !pars_[pit].isLineSeparator(body_pos - 1)))
+           (body_pos > end || !par.isLineSeparator(body_pos - 1)))
                body_pos = 0;
 
        // check for empty row
@@ -1052,23 +1108,23 @@ pos_type LyXText::getColumnNearX(par_type pit,
                last_tmpx = tmpx;
                if (body_pos > 0 && c == body_pos - 1) {
                        tmpx += r.label_hfill +
-                               font_metrics::width(layout->labelsep, getLabelFont(pit));
-                       if (pars_[pit].isLineSeparator(body_pos - 1))
-                               tmpx -= singleWidth(pit, body_pos - 1);
+                               font_metrics::width(layout->labelsep, getLabelFont(par));
+                       if (par.isLineSeparator(body_pos - 1))
+                               tmpx -= singleWidth(par, body_pos - 1);
                }
 
-               if (hfillExpansion(pars_[pit], row, c)) {
-                       tmpx += singleWidth(pit, c);
+               if (hfillExpansion(par, row, c)) {
+                       tmpx += singleWidth(par, c);
                        if (c >= body_pos)
                                tmpx += r.hfill;
                        else
                                tmpx += r.label_hfill;
-               } else if (pars_[pit].isSeparator(c)) {
-                       tmpx += singleWidth(pit, c);
+               } else if (par.isSeparator(c)) {
+                       tmpx += singleWidth(par, c);
                        if (c >= body_pos)
                                tmpx += r.separator;
                } else {
-                       tmpx += singleWidth(pit, c);
+                       tmpx += singleWidth(par, c);
                }
                ++vc;
        }
@@ -1083,11 +1139,11 @@ pos_type LyXText::getColumnNearX(par_type pit,
        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() == pars_[pit].size();
+       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(pars_[pit]) : false;
+       bool const rtl = lastrow ? isRTL(par) : false;
        if (lastrow &&
                 ((rtl  &&  left_side && vc == row.pos() && x < tmpx - 5) ||
                  (!rtl && !left_side && vc == end  && x > tmpx + 5)))
@@ -1101,15 +1157,15 @@ pos_type LyXText::getColumnNearX(par_type pit,
                bool const rtl = (bidi.level(c) % 2 == 1);
                if (left_side == rtl) {
                        ++c;
-                       boundary = bidi.isBoundary(*bv()->buffer(), pars_[pit], c);
+                       boundary = bidi.isBoundary(*bv()->buffer(), par, c);
                }
        }
 
-       if (row.pos() < end && c >= end && pars_[pit].isNewline(end - 1)) {
+       if (row.pos() < end && c >= end && par.isNewline(end - 1)) {
                if (bidi.level(end -1) % 2 == 0)
-                       tmpx -= singleWidth(pit, end - 1);
+                       tmpx -= singleWidth(par, end - 1);
                else
-                       tmpx += singleWidth(pit, end - 1);
+                       tmpx += singleWidth(par, end - 1);
                c = end - 1;
        }
 
@@ -1118,14 +1174,37 @@ pos_type LyXText::getColumnNearX(par_type pit,
 }
 
 
+// y is relative to this LyXText's top
+// this is only used in the two functions below
+Row const & LyXText::getRowNearY(int y, pit_type & pit) const
+{
+       BOOST_ASSERT(!paragraphs().empty());
+       BOOST_ASSERT(!paragraphs().begin()->rows.empty());
+       pit_type const pend = paragraphs().size() - 1;
+       pit = 0;
+       while (int(pars_[pit].y + pars_[pit].height) < y && pit != pend)
+               ++pit;
+
+       RowList::iterator rit = pars_[pit].rows.end();
+       RowList::iterator const rbegin = pars_[pit].rows.begin();
+       do {
+               --rit;
+       } while (rit != rbegin && int(pars_[pit].y + rit->y_offset()) > y);
+
+       return *rit;
+}
+
+
 // x,y are absolute coordinates
+// sets cursor only within this LyXText
 void LyXText::setCursorFromCoordinates(LCursor & cur, int x, int y)
 {
        x -= xo_;
        y -= yo_;
-       par_type pit;
-       Row const & row = *getRowNearY(y, pit);
-       lyxerr << "setCursorFromCoordinates:: hit row at: " << row.pos() << endl;
+       pit_type pit;
+       Row const & row = getRowNearY(y, pit);
+       lyxerr[Debug::DEBUG] << "setCursorFromCoordinates:: hit row at: "
+                            << row.pos() << endl;
        bool bound = false;
        int xx = x + xo_; // getRowNearX get absolute x coords
        pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
@@ -1134,15 +1213,16 @@ void LyXText::setCursorFromCoordinates(LCursor & cur, int x, int y)
 
 
 // x,y are absolute screen coordinates
-InsetBase * LyXText::editXY(LCursor & cur, int x, int y)
+// sets cursor recursively descending into nested editable insets
+InsetBase * LyXText::editXY(LCursor & cur, int x, int y) const
 {
-       par_type pit;
-       Row const & row = *getRowNearY(y - yo_, pit);
+       pit_type pit;
+       Row const & row = getRowNearY(y - yo_, pit);
        bool bound = false;
 
        int xx = x; // is modified by getColumnNearX
        pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
-       cur.par() = pit;
+       cur.pit() = pit;
        cur.pos() = pos;
        cur.boundary() = bound;
 
@@ -1182,18 +1262,18 @@ void LyXText::cursorLeft(LCursor & cur)
 {
        if (cur.pos() != 0) {
                bool boundary = cur.boundary();
-               setCursor(cur, cur.par(), cur.pos() - 1, true, false);
+               setCursor(cur, cur.pit(), cur.pos() - 1, true, false);
                if (!checkAndActivateInset(cur, false)) {
                        if (false && !boundary &&
                                        bidi.isBoundary(cur.buffer(), cur.paragraph(), cur.pos() + 1))
-                               setCursor(cur, cur.par(), cur.pos() + 1, true, true);
+                               setCursor(cur, cur.pit(), cur.pos() + 1, true, true);
                }
                return;
        }
 
-       if (cur.par() != 0) {
+       if (cur.pit() != 0) {
                // steps into the paragraph above
-               setCursor(cur, cur.par() - 1, getPar(cur.par() - 1).size());
+               setCursor(cur, cur.pit() - 1, getPar(cur.pit() - 1).size());
        }
 }
 
@@ -1201,22 +1281,22 @@ void LyXText::cursorLeft(LCursor & cur)
 void LyXText::cursorRight(LCursor & cur)
 {
        if (false && cur.boundary()) {
-               setCursor(cur, cur.par(), cur.pos(), true, false);
+               setCursor(cur, cur.pit(), cur.pos(), true, false);
                return;
        }
 
        if (cur.pos() != cur.lastpos()) {
                if (!checkAndActivateInset(cur, true)) {
-                       setCursor(cur, cur.par(), cur.pos() + 1, true, false);
+                       setCursor(cur, cur.pit(), cur.pos() + 1, true, false);
                        if (false && bidi.isBoundary(cur.buffer(), cur.paragraph(),
                                                         cur.pos()))
-                               setCursor(cur, cur.par(), cur.pos(), true, true);
+                               setCursor(cur, cur.pit(), cur.pos(), true, true);
                }
                return;
        }
 
-       if (cur.par() != cur.lastpar())
-               setCursor(cur, cur.par() + 1, 0);
+       if (cur.pit() != cur.lastpit())
+               setCursor(cur, cur.pit() + 1, 0);
 }
 
 
@@ -1253,18 +1333,18 @@ void LyXText::cursorDown(LCursor & cur)
 void LyXText::cursorUpParagraph(LCursor & cur)
 {
        if (cur.pos() > 0)
-               setCursor(cur, cur.par(), 0);
-       else if (cur.par() != 0)
-               setCursor(cur, cur.par() - 1, 0);
+               setCursor(cur, cur.pit(), 0);
+       else if (cur.pit() != 0)
+               setCursor(cur, cur.pit() - 1, 0);
 }
 
 
 void LyXText::cursorDownParagraph(LCursor & cur)
 {
-       if (cur.par() != cur.lastpar())
-               setCursor(cur, cur.par() + 1, 0);
+       if (cur.pit() != cur.lastpit())
+               setCursor(cur, cur.pit() + 1, 0);
        else
-               setCursor(cur, cur.par(), cur.lastpos());
+               setCursor(cur, cur.pit(), cur.lastpos());
 }
 
 
@@ -1274,7 +1354,7 @@ void LyXText::fixCursorAfterDelete(CursorSlice & cur, CursorSlice const & where)
 {
        // do notheing if cursor is not in the paragraph where the
        // deletion occured,
-       if (cur.par() != where.par())
+       if (cur.pit() != where.pit())
                return;
 
        // if cursor position is after the deletion place update it
@@ -1295,8 +1375,8 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
        if (cur.selection())
                return false;
 
-       //lyxerr << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
-       Paragraph const & oldpar = pars_[old.par()];
+       //lyxerr[Debug::DEBUG] << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
+       Paragraph const & oldpar = pars_[old.pit()];
 
        // We allow all kinds of "mumbo-jumbo" when freespacing.
        if (oldpar.isFreeSpacing())
@@ -1326,14 +1406,14 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
        // MISSING
 
        // If the chars around the old cursor were spaces, delete one of them.
-       if (old.par() != cur.par() || old.pos() != cur.pos()) {
+       if (old.pit() != cur.pit() || old.pos() != cur.pos()) {
 
                // Only if the cursor has really moved.
                if (old.pos() > 0
                    && old.pos() < oldpar.size()
                    && oldpar.isLineSeparator(old.pos())
                    && oldpar.isLineSeparator(old.pos() - 1)) {
-                       pars_[old.par()].erase(old.pos() - 1);
+                       pars_[old.pit()].erase(old.pos() - 1);
 #ifdef WITH_WARNINGS
 #warning This will not work anymore when we have multiple views of the same buffer
 // In this case, we will have to correct also the cursors held by
@@ -1342,14 +1422,16 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
 #endif
                        // correct all cursor parts
                        fixCursorAfterDelete(cur.top(), old.top());
+#ifdef WITH_WARNINGS
 #warning DEPM, look here
+#endif
                        //fixCursorAfterDelete(cur.anchor(), old.top());
                        return false;
                }
        }
 
        // only do our magic if we changed paragraph
-       if (old.par() == cur.par())
+       if (old.pit() == cur.pit())
                return false;
 
        // don't delete anything if this is the ONLY paragraph!
@@ -1371,23 +1453,25 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
                deleted = true;
 
                bool selection_position_was_oldcursor_position =
-                       cur.anchor().par() == old.par() && cur.anchor().pos() == old.pos();
+                       cur.anchor().pit() == old.pit() && cur.anchor().pos() == old.pos();
 
                // This is a bit of a overkill. We change the old and the cur par
                // at max, certainly not everything in between...
-               recUndo(old.par(), cur.par());
+               recUndo(old.pit(), cur.pit());
 
                // Delete old par.
-               pars_.erase(pars_.begin() + old.par());
+               pars_.erase(pars_.begin() + old.pit());
 
                // Update cursor par offset if necessary.
                // Some 'iterator registration' would be nice that takes care of
                // such events. Maybe even signal/slot?
-               if (cur.par() > old.par())
-                       --cur.par();
+               if (cur.pit() > old.pit())
+                       --cur.pit();
+#ifdef WITH_WARNINGS
 #warning DEPM, look here
-//             if (cur.anchor().par() > old.par())
-//                     --cur.anchor().par();
+#endif
+//             if (cur.anchor().pit() > old.pit())
+//                     --cur.anchor().pit();
 
                if (selection_position_was_oldcursor_position) {
                        // correct selection
@@ -1398,7 +1482,7 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
        if (deleted)
                return true;
 
-       if (pars_[old.par()].stripLeadingSpaces())
+       if (pars_[old.pit()].stripLeadingSpaces())
                cur.resetAnchor();
 
        return false;
@@ -1411,13 +1495,13 @@ ParagraphList & LyXText::paragraphs() const
 }
 
 
-void LyXText::recUndo(par_type first, par_type last) const
+void LyXText::recUndo(pit_type first, pit_type last) const
 {
        recordUndo(bv()->cursor(), Undo::ATOMIC, first, last);
 }
 
 
-void LyXText::recUndo(par_type par) const
+void LyXText::recUndo(pit_type par) const
 {
        recordUndo(bv()->cursor(), Undo::ATOMIC, par, par);
 }