]> git.lyx.org Git - lyx.git/blobdiff - src/text3.C
the convert patch
[lyx.git] / src / text3.C
index 6b9213a9dae673772feb1a4023d45389df7d9312..daf8a9ab7216eb6825835c9f4273700d5af815b1 100644 (file)
@@ -23,6 +23,7 @@
 #include "bufferparams.h"
 #include "BufferView.h"
 #include "cursor.h"
+#include "coordcache.h"
 #include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
@@ -53,7 +54,7 @@
 
 #include "support/lstrings.h"
 #include "support/lyxlib.h"
-#include "support/tostr.h"
+#include "support/convert.h"
 
 #include "mathed/math_hullinset.h"
 #include "mathed/math_macrotemplate.h"
@@ -69,8 +70,6 @@ using lyx::cap::pasteSelection;
 using lyx::cap::replaceSelection;
 
 using lyx::support::isStrUnsignedInt;
-using lyx::support::strToUnsignedInt;
-using lyx::support::atoi;
 using lyx::support::token;
 
 using std::endl;
@@ -82,10 +81,6 @@ using std::vector;
 
 extern string current_layout;
 
-// the selection possible is needed, that only motion events are
-// used, where the button press event was on the drawing area too
-bool selection_possible = false;
-
 
 namespace {
 
@@ -107,7 +102,7 @@ namespace {
                                        text->bidi.isBoundary(cur.buffer(), par,
                                                        cur.pos(),
                                                        text->real_current_font))
-                               text->setCursor(cur, cur.par(), cur.pos(),
+                               text->setCursor(cur, cur.pit(), cur.pos(),
                                                false, !cur.boundary());
                }
        }
@@ -142,15 +137,17 @@ namespace {
                        BOOST_ASSERT(old_pos == cur.pos());
                        cur.nextInset()->edit(cur, true);
                        cur.dispatch(FuncRequest(LFUN_MATH_MUTATE, "simple"));
-                       // don't do that also for LFUN_MATH_MODE unless you want end up with
-                       // always changing to mathrm when opening an inlined inset
-                       // -- I really hate "LyXfunc overloading"...
+                       // don't do that also for LFUN_MATH_MODE
+                       // unless you want end up with always changing
+                       // to mathrm when opening an inlined inset --
+                       // I really hate "LyXfunc overloading"...
                        if (display)
                                cur.dispatch(FuncRequest(LFUN_MATH_DISPLAY));
                        cur.dispatch(FuncRequest(LFUN_INSERT_MATH, cmd.argument));
                } else {
-                       // create a macro if we see "\\newcommand" somewhere, and an ordinary
-                       // formula otherwise
+                       // create a macro if we see "\\newcommand"
+                       // somewhere, and an ordinary formula
+                       // otherwise
                        cutSelection(cur, true, true);
                        if (sel.find("\\newcommand") == string::npos
                            && sel.find("\\def") == string::npos)
@@ -183,48 +180,12 @@ string const freefont2string()
 
 }
 
-
-//takes absolute x,y coordinates
-InsetBase * LyXText::checkInsetHit(int x, int y) const 
-{
-       par_type pit;
-       par_type end;
-
-       getParsInRange(paragraphs(),
-                      bv()->top_y() - yo_,
-                      bv()->top_y() - yo_ + bv()->workHeight(),
-                      pit, end);
-
-       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 0
-                       lyxerr << "examining inset " << inset
-                               << " xo: " << inset->xo() << "..." << inset->xo() + inset->width()
-                               << " yo: " << inset->yo() - inset->ascent() << "..."
-                               << inset->yo() + inset->descent() << endl;
-#endif
-                       if (inset->covers(x, y)) {
-                               lyxerr << "Hit inset: " << inset << endl;
-                               return inset;
-                       }
-               }
-       }
-       //lyxerr << "No inset hit. " << endl;
-       return 0;
-}
-
-
 bool LyXText::gotoNextInset(LCursor & cur,
-       vector<InsetOld_code> const & codes, string const & contents)
+       vector<InsetBase_code> const & codes, string const & contents)
 {
        BOOST_ASSERT(this == cur.text());
-       par_type end = paragraphs().size();
-       par_type pit = cur.par();
+       pit_type end = paragraphs().size();
+       pit_type pit = cur.pit();
        pos_type pos = cur.pos();
 
        InsetBase * inset;
@@ -253,7 +214,7 @@ bool LyXText::gotoNextInset(LCursor & cur,
 
 
 void LyXText::gotoInset(LCursor & cur,
-       vector<InsetOld_code> const & codes, bool same_content)
+       vector<InsetBase_code> const & codes, bool same_content)
 {
        cur.clearSelection();
 
@@ -268,9 +229,9 @@ void LyXText::gotoInset(LCursor & cur,
        }
 
        if (!gotoNextInset(cur, codes, contents)) {
-               if (cur.pos() || cur.par() != 0) {
+               if (cur.pos() || cur.pit() != 0) {
                        CursorSlice tmp = cur.top();
-                       cur.par() = 0;
+                       cur.pit() = 0;
                        cur.pos() = 0;
                        if (!gotoNextInset(cur, codes, contents)) {
                                cur.top() = tmp;
@@ -284,22 +245,23 @@ void LyXText::gotoInset(LCursor & cur,
 }
 
 
-void LyXText::gotoInset(LCursor & cur, InsetOld_code code, bool same_content)
+void LyXText::gotoInset(LCursor & cur, InsetBase_code code, bool same_content)
 {
-       gotoInset(cur, vector<InsetOld_code>(1, code), same_content);
+       gotoInset(cur, vector<InsetBase_code>(1, code), same_content);
 }
 
 
 void LyXText::cursorPrevious(LCursor & cur)
 {
        pos_type cpos = cur.pos();
-       lyx::par_type cpar = cur.par();
+       lyx::pit_type cpar = cur.pit();
 
        int x = cur.x_target();
-       int y = cur.bv().top_y();
-       setCursorFromCoordinates(cur, x, y);
 
-       if (cpar == cur.par() && cpos == cur.pos()) {
+       setCursorFromCoordinates(cur, x, 0);
+       cursorUp(cur);
+
+       if (cpar == cur.pit() && cpos == cur.pos()) {
                // we have a row which is taller than the workarea. The
                // simplest solution is to move to the previous row instead.
                cursorUp(cur);
@@ -313,13 +275,13 @@ void LyXText::cursorPrevious(LCursor & cur)
 void LyXText::cursorNext(LCursor & cur)
 {
        pos_type cpos = cur.pos();
-       lyx::par_type cpar = cur.par();
+       lyx::pit_type cpar = cur.pit();
 
        int x = cur.x_target();
-       int y = cur.bv().top_y() + cur.bv().workHeight();
-       setCursorFromCoordinates(cur, x, y);
+       setCursorFromCoordinates(cur, x, cur.bv().workHeight() - 1);
+       cursorDown(cur);
 
-       if (cpar == cur.par() && cpos == cur.pos()) {
+       if (cpar == cur.pit() && cpos == cur.pos()) {
                // we have a row which is taller than the workarea. The
                // simplest solution is to move to the next row instead.
                cursorDown(cur);
@@ -336,6 +298,7 @@ void specialChar(LCursor & cur, InsetSpecialChar::Kind kind)
 {
        lyx::cap::replaceSelection(cur);
        cur.insert(new InsetSpecialChar(kind));
+       cur.posRight();
 }
 
 
@@ -361,6 +324,15 @@ void doInsertInset(LCursor & cur, LyXText * text,
                cur.bv().owner()->dispatch(FuncRequest(LFUN_PASTE));
 }
 
+
+void update(LCursor & cur)
+{
+       //we don't call update(true, false) directly to save a metrics call
+       if (cur.bv().fitCursor())
+               cur.bv().update(false, true);
+}
+
+
 } // anon namespace
 
 
@@ -396,11 +368,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                bool start = !par.params().startOfAppendix();
 
                // ensure that we have only one start_of_appendix in this document
-               for (par_type tmp = 0, end = pars_.size(); tmp != end; ++tmp) {
+               for (pit_type tmp = 0, end = pars_.size(); tmp != end; ++tmp) {
                        if (pars_[tmp].params().startOfAppendix()) {
                                recUndo(tmp);
                                pars_[tmp].params().startOfAppendix(false);
-                               redoParagraph(tmp);
                                break;
                        }
                }
@@ -410,7 +381,6 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
                // we can set the refreshing parameters now
                updateCounters();
-               redoParagraph(cur);
                break;
        }
 
@@ -454,7 +424,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                finishChange(cur, false);
                break;
 
-       case LFUN_BEGINNINGBUF: 
+       case LFUN_BEGINNINGBUF:
                if (cur.size() == 1) {
                        if (!cur.mark())
                                cur.clearSelection();
@@ -465,6 +435,17 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                }
                break;
 
+       case LFUN_BEGINNINGBUFSEL:
+               if (cur.size() == 1) {
+                       if (!cur.selection())
+                               cur.resetAnchor();
+                       cursorTop(cur);
+                       finishChange(cur, true);
+               } else {
+                       cur.undispatched();
+               }
+               break;
+
        case LFUN_ENDBUF:
                if (cur.size() == 1) {
                        if (!cur.mark())
@@ -476,6 +457,17 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                }
                break;
 
+       case LFUN_ENDBUFSEL:
+               if (cur.size() == 1) {
+                       if (!cur.selection())
+                               cur.resetAnchor();
+                       cursorBottom(cur);
+                       finishChange(cur, true);
+               } else {
+                       cur.undispatched();
+               }
+               break;
+
        case LFUN_RIGHT:
                moving = true;
        case LFUN_RIGHTSEL:
@@ -509,6 +501,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
        case LFUN_UP:
                moving = true;
        case LFUN_UPSEL:
+               update(cur);
                //lyxerr << "handle LFUN_UP[SEL]:\n" << cur << endl;
                cur.selHandle(cmd.action == LFUN_UPSEL);
                cursorUp(cur);
@@ -521,6 +514,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
        case LFUN_DOWN:
                moving = true;
        case LFUN_DOWNSEL:
+               update(cur);
                //lyxerr << "handle LFUN_DOWN[SEL]:\n" << cur << endl;
                cur.selHandle(cmd.action == LFUN_DOWNSEL);
                cursorDown(cur);
@@ -545,6 +539,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_PRIORSEL:
+               update(cur);
                if (!cur.selection())
                        cur.resetAnchor();
                cursorPrevious(cur);
@@ -552,6 +547,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_NEXTSEL:
+               update(cur);
                if (!cur.selection())
                        cur.resetAnchor();
                cursorNext(cur);
@@ -559,6 +555,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_HOMESEL:
+               update(cur);
                if (!cur.selection())
                        cur.resetAnchor();
                cursorHome(cur);
@@ -566,6 +563,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_ENDSEL:
+               update(cur);
                if (!cur.selection())
                        cur.resetAnchor();
                cursorEnd(cur);
@@ -615,11 +613,12 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_PRIOR:
+               update(cur);
                moving = true;
                if (!cur.mark())
                        cur.clearSelection();
                finishChange(cur, false);
-               if (cur.par() == 0 && cur.textRow().pos() == 0) {
+               if (cur.pit() == 0 && cur.textRow().pos() == 0) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_UP);
                } else {
@@ -628,11 +627,12 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_NEXT:
+               update(cur);
                moving = true;
                if (!cur.mark())
                        cur.clearSelection();
                finishChange(cur, false);
-               if (cur.par() == cur.lastpar()
+               if (cur.pit() == cur.lastpit()
                          && cur.textRow().endpos() == cur.lastpos()) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_DOWN);
@@ -660,6 +660,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                if (cur.pos() > cur.paragraph().beginOfBody()) {
                        lyx::cap::replaceSelection(cur);
                        cur.insert(new InsetNewline);
+                       cur.posRight();
                        moveCursor(cur, false);
                }
                break;
@@ -750,15 +751,15 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
        case LFUN_PARAGRAPH_SPACING: {
                Paragraph & par = cur.paragraph();
                Spacing::Space cur_spacing = par.params().spacing().getSpace();
-               float cur_value = 1.0;
+               string cur_value = "1.0";
                if (cur_spacing == Spacing::Other)
-                       cur_value = par.params().spacing().getValue();
+                       cur_value = par.params().spacing().getValueAsString();
 
                istringstream is(cmd.argument);
                string tmp;
                is >> tmp;
                Spacing::Space new_spacing = cur_spacing;
-               float new_value = cur_value;
+               string new_value = cur_value;
                if (tmp.empty()) {
                        lyxerr << "Missing argument to `paragraph-spacing'"
                               << endl;
@@ -770,10 +771,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                        new_spacing = Spacing::Double;
                } else if (tmp == "other") {
                        new_spacing = Spacing::Other;
-                       float tmpval = 0.0;
+                       string tmpval = "0.0";
                        is >> tmpval;
                        lyxerr << "new_value = " << tmpval << endl;
-                       if (tmpval != 0.0)
+                       if (tmpval != "0.0")
                                new_value = tmpval;
                } else if (tmp == "default") {
                        new_spacing = Spacing::Default;
@@ -781,10 +782,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                        lyxerr << _("Unknown spacing argument: ")
                               << cmd.argument << endl;
                }
-               if (cur_spacing != new_spacing || cur_value != new_value) {
+               if (cur_spacing != new_spacing || cur_value != new_value)
                        par.params().spacing(Spacing(new_spacing, new_value));
-                       redoParagraph(cur);
-               }
                break;
        }
 
@@ -835,8 +834,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
        case LFUN_SPACE_INSERT:
                if (cur.paragraph().layout()->free_spacing)
                        insertChar(cur, ' ');
-               else
+               else {
                        doInsertInset(cur, this, cmd, false, false);
+                       cur.posRight();
+               }
                moveCursor(cur, false);
                break;
 
@@ -874,7 +875,6 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
        case LFUN_TRANSPOSE_CHARS:
                recordUndo(cur);
-               redoParagraph(cur);
                break;
 
        case LFUN_PASTE:
@@ -884,7 +884,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 #warning FIXME Check if the arg is in the domain of available selections.
 #endif
                if (isStrUnsignedInt(cmd.argument))
-                       pasteSelection(cur, strToUnsignedInt(cmd.argument));
+                       pasteSelection(cur, convert<unsigned int>(cmd.argument));
                else
                        pasteSelection(cur, 0);
                cur.clearSelection(); // bug 393
@@ -903,8 +903,8 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_GETXY:
-               cur.message(tostr(cursorX(cur.top())) + ' '
-                         + tostr(cursorY(cur.top())));
+               cur.message(convert<string>(cursorX(cur.top())) + ' '
+                         + convert<string>(cursorY(cur.top())));
                break;
 
        case LFUN_SETXY: {
@@ -969,10 +969,10 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                bool change_layout = (current_layout != layout);
 
                if (!change_layout && cur.selection() &&
-                       cur.selBegin().par() != cur.selEnd().par())
+                       cur.selBegin().pit() != cur.selEnd().pit())
                {
-                       par_type spit = cur.selBegin().par();
-                       par_type epit = cur.selEnd().par() + 1;
+                       pit_type spit = cur.selBegin().pit();
+                       pit_type epit = cur.selEnd().pit() + 1;
                        while (spit != epit) {
                                if (pars_[spit].layout()->name() != current_layout) {
                                        change_layout = true;
@@ -1012,7 +1012,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_REFERENCE_GOTO: {
-               vector<InsetOld_code> tmp;
+               vector<InsetBase_code> tmp;
                tmp.push_back(InsetBase::LABEL_CODE);
                tmp.push_back(InsetBase::REF_CODE);
                gotoInset(cur, tmp, true);
@@ -1045,7 +1045,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                                cur.insert(new InsetQuotes(c,
                                    bufparams.quotes_language,
                                    InsetQuotes::DoubleQ));
-                       else 
+                       else
                                cur.insert(new InsetQuotes(c, bufparams));
                        cur.posRight();
                }
@@ -1078,7 +1078,6 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
        case LFUN_MOUSE_TRIPLE:
                if (cmd.button() == mouse_button::button1) {
-                       selection_possible = true;
                        cursorHome(cur);
                        cur.resetAnchor();
                        cursorEnd(cur);
@@ -1089,54 +1088,16 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
        case LFUN_MOUSE_DOUBLE:
                if (cmd.button() == mouse_button::button1) {
-                       selection_possible = true;
                        selectWord(cur, lyx::WHOLE_WORD_STRICT);
                        bv->haveSelection(cur.selection());
                }
                break;
 
-       case LFUN_MOUSE_MOTION: {
-               // Only use motion with button 1
-               //if (cmd.button() != mouse_button::button1)
-               //      return false;
-               // We want to use only motion events for which
-               // the button press event was on the drawing area too.
-               if (!selection_possible) {
-                       lyxerr[Debug::ACTION] << "BufferView::Pimpl::"
-                               "dispatch: no selection possible\n";
-                       lyxerr << "BufferView::Pimpl::dispatch: no selection possible\n";
-                       break;
-               }
-
-               // ignore motions deeper nested than the real anchor
-               LCursor & bvcur = cur.bv().cursor();
-               if (bvcur.selection() && bvcur.anchor_.size() < cur.size())
-                       break;
-
-               CursorSlice old = cur.top();
-               setCursorFromCoordinates(cur, cmd.x, cmd.y);
-
-               // This is to allow jumping over large insets
-               // FIXME: shouldn't be top-text-specific
-               if (isMainText() && cur.top() == old) {
-                       if (cmd.y - bv->top_y() >= bv->workHeight())
-                               cursorDown(cur);
-                       else if (cmd.y - bv->top_y() < 0)
-                               cursorUp(cur);
-               }
-
-               // don't set anchor_
-               bv->cursor().setCursor(cur, true);
-               lyxerr << "MOTION: " << bv->cursor() << endl;
-               break;
-       }
-
        // Single-click on work area
        case LFUN_MOUSE_PRESS: {
                // Right click on a footnote flag opens float menu
                if (cmd.button() == mouse_button::button3) {
                        cur.clearSelection();
-                       selection_possible = false;
                        break;
                }
 
@@ -1150,15 +1111,13 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                        paste_internally = true;
                }
 
-               selection_possible = true;
-
                // Clear the selection
                cur.clearSelection();
 
                setCursorFromCoordinates(cur, cmd.x, cmd.y);
                cur.resetAnchor();
                finishUndo();
-               cur.x_target() = cursorX(cur.top());
+               cur.setTargetX();
 
                // Has the cursor just left the inset?
                if (bv->cursor().inMathed() && !cur.inMathed())
@@ -1167,10 +1126,6 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                // Set cursor here.
                bv->cursor() = cur;
 
-               // Don't allow selection after a big jump.
-               if (bv->fitCursor())
-                       selection_possible = false;
-
                // Insert primary selection with middle mouse
                // if there is a local selection in the current buffer,
                // insert this
@@ -1179,15 +1134,53 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                                bv->owner()->dispatch(FuncRequest(LFUN_PASTE));
                        else
                                bv->owner()->dispatch(FuncRequest(LFUN_PASTESELECTION, "paragraph"));
-                       selection_possible = false;
                }
 
                break;
        }
 
-       case LFUN_MOUSE_RELEASE: {
-               selection_possible = false;
+       case LFUN_MOUSE_MOTION: {
+               // Only use motion with button 1
+               //if (cmd.button() != mouse_button::button1)
+               //      return false;
+
+               // ignore motions deeper nested than the real anchor
+               LCursor & bvcur = cur.bv().cursor();
+               if (bvcur.anchor_.hasPart(cur)) {
+                       CursorSlice old = bvcur.top();
+
+                       int const wh = bv->workHeight();
+                       int const y = std::max(0, std::min(wh - 1, cmd.y));
 
+                       setCursorFromCoordinates(cur, cmd.x, y);
+                       cur.x_target() = cmd.x;
+                       if (cmd.y >= wh)
+                               cursorDown(cur);
+                       else if (cmd.y < 0)
+                               cursorUp(cur);
+                       // This is to allow jumping over large insets
+                       if (cur.top() == old) {
+                               if (cmd.y >= wh)
+                                       cursorDown(cur);
+                               else if (cmd.y < 0)
+                                       cursorUp(cur);
+                       }
+
+                       if (cur.top() == old)
+                               cur.noUpdate();
+                       else {
+                               // don't set anchor_
+                               bvcur.setCursor(cur);
+                               bvcur.selection() = true;
+                               lyxerr << "MOTION: " << bv->cursor() << endl;
+                       }
+
+               } else
+                       cur.undispatched();
+               break;
+       }
+
+       case LFUN_MOUSE_RELEASE: {
                if (cmd.button() == mouse_button::button2)
                        break;
 
@@ -1323,7 +1316,11 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
        case LFUN_MATH_IMPORT_SELECTION:
        case LFUN_MATH_MODE:
-               mathDispatch(cur, cmd, false);
+               if (cmd.argument == "on")
+                       // don't pass "on" as argument
+                       mathDispatch(cur, FuncRequest(LFUN_MATH_MODE), false);
+               else
+                       mathDispatch(cur, cmd, false);
                break;
 
        case LFUN_MATH_MACRO:
@@ -1332,7 +1329,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                else {
                        string s = cmd.argument;
                        string const s1 = token(s, ' ', 1);
-                       int const nargs = s1.empty() ? 0 : atoi(s1);
+                       int const nargs = s1.empty() ? 0 : convert<int>(s1);
                        string const s2 = token(s, ' ', 2);
                        string const type = s2.empty() ? "newcommand" : s2;
                        cur.insert(new MathMacroTemplate(token(s, ' ', 0), nargs, s2));
@@ -1340,6 +1337,14 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                }
                break;
 
+       // passthrough hat and underscore outside mathed:
+       case LFUN_SUBSCRIPT:
+               mathDispatch(cur, FuncRequest(LFUN_SELFINSERT, "_"), false);
+               break;
+       case LFUN_SUPERSCRIPT:
+               mathDispatch(cur, FuncRequest(LFUN_SELFINSERT, "^"), false);
+               break;
+
        case LFUN_INSERT_MATH:
        case LFUN_INSERT_MATRIX:
        case LFUN_MATH_DELIM: {
@@ -1443,21 +1448,21 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
        }
 
        case LFUN_FINISHED_LEFT:
-               lyxerr << "handle LFUN_FINISHED_LEFT:\n" << cur << endl;
+               lyxerr[Debug::DEBUG] << "handle LFUN_FINISHED_LEFT:\n" << cur << endl;
                break;
 
        case LFUN_FINISHED_RIGHT:
-               lyxerr << "handle LFUN_FINISHED_RIGHT:\n" << cur << endl;
+               lyxerr[Debug::DEBUG] << "handle LFUN_FINISHED_RIGHT:\n" << cur << endl;
                ++cur.pos();
                break;
 
        case LFUN_FINISHED_UP:
-               lyxerr << "handle LFUN_FINISHED_UP:\n" << cur << endl;
+               lyxerr[Debug::DEBUG] << "handle LFUN_FINISHED_UP:\n" << cur << endl;
                cursorUp(cur);
                break;
 
        case LFUN_FINISHED_DOWN:
-               lyxerr << "handle LFUN_FINISHED_DOWN:\n" << cur << endl;
+               lyxerr[Debug::DEBUG] << "handle LFUN_FINISHED_DOWN:\n" << cur << endl;
                cursorDown(cur);
                break;
 
@@ -1479,7 +1484,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                InsetBase & inset = cur.inset();
                bool const accept = !inset.forceDefaultParagraphs(&inset);
 
-               data = "update " + tostr(accept) + '\n' + data;
+               data = "update " + convert<string>(accept) + '\n' + data;
                bv->owner()->getDialogs().update("paragraph", data);
                break;
        }
@@ -1523,6 +1528,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
                        setLayout(cur, tclass.defaultLayoutName());
                        setParagraph(cur, Spacing(), LYX_ALIGN_LAYOUT, string(), 0);
                        insertInset(cur, new InsetFloatList(cmd.argument));
+                       cur.posRight();
                } else {
                        lyxerr << "Non-existent float type: "
                               << cmd.argument << endl;
@@ -1607,7 +1613,7 @@ void LyXText::dispatch(LCursor & cur, FuncRequest & cmd)
 
 
 bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
-       FuncStatus & flag) const
+                       FuncStatus & flag) const
 {
        BOOST_ASSERT(cur.text() == this);
        LyXFont const & font = real_current_font;
@@ -1624,7 +1630,8 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
                break;
 
        case LFUN_INSET_OPTARG:
-               enable = cur.paragraph().layout()->optionalargs;
+               enable = numberOfOptArgs(cur.paragraph())
+                       < cur.paragraph().layout()->optionalargs;
                break;
 
        case LFUN_APPENDIX:
@@ -1633,117 +1640,117 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
 
 #if 0
        // the functions which insert insets
-       InsetOld::Code code = InsetOld::NO_CODE;
+       InsetBase::Code code = InsetBase::NO_CODE;
        switch (cmd.action) {
        case LFUN_DIALOG_SHOW_NEW_INSET:
                if (cmd.argument == "bibitem")
-                       code = InsetOld::BIBITEM_CODE;
+                       code = InsetBase::BIBITEM_CODE;
                else if (cmd.argument == "bibtex")
-                       code = InsetOld::BIBTEX_CODE;
+                       code = InsetBase::BIBTEX_CODE;
                else if (cmd.argument == "box")
-                       code = InsetOld::BOX_CODE;
+                       code = InsetBase::BOX_CODE;
                else if (cmd.argument == "branch")
-                       code = InsetOld::BRANCH_CODE;
+                       code = InsetBase::BRANCH_CODE;
                else if (cmd.argument == "citation")
-                       code = InsetOld::CITE_CODE;
+                       code = InsetBase::CITE_CODE;
                else if (cmd.argument == "ert")
-                       code = InsetOld::ERT_CODE;
+                       code = InsetBase::ERT_CODE;
                else if (cmd.argument == "external")
-                       code = InsetOld::EXTERNAL_CODE;
+                       code = InsetBase::EXTERNAL_CODE;
                else if (cmd.argument == "float")
-                       code = InsetOld::FLOAT_CODE;
+                       code = InsetBase::FLOAT_CODE;
                else if (cmd.argument == "graphics")
-                       code = InsetOld::GRAPHICS_CODE;
+                       code = InsetBase::GRAPHICS_CODE;
                else if (cmd.argument == "include")
-                       code = InsetOld::INCLUDE_CODE;
+                       code = InsetBase::INCLUDE_CODE;
                else if (cmd.argument == "index")
-                       code = InsetOld::INDEX_CODE;
+                       code = InsetBase::INDEX_CODE;
                else if (cmd.argument == "label")
-                       code = InsetOld::LABEL_CODE;
+                       code = InsetBase::LABEL_CODE;
                else if (cmd.argument == "note")
-                       code = InsetOld::NOTE_CODE;
+                       code = InsetBase::NOTE_CODE;
                else if (cmd.argument == "ref")
-                       code = InsetOld::REF_CODE;
+                       code = InsetBase::REF_CODE;
                else if (cmd.argument == "toc")
-                       code = InsetOld::TOC_CODE;
+                       code = InsetBase::TOC_CODE;
                else if (cmd.argument == "url")
-                       code = InsetOld::URL_CODE;
+                       code = InsetBase::URL_CODE;
                else if (cmd.argument == "vspace")
-                       code = InsetOld::VSPACE_CODE;
+                       code = InsetBase::VSPACE_CODE;
                else if (cmd.argument == "wrap")
-                       code = InsetOld::WRAP_CODE;
+                       code = InsetBase::WRAP_CODE;
                break;
 
        case LFUN_INSET_ERT:
-               code = InsetOld::ERT_CODE;
+               code = InsetBase::ERT_CODE;
                break;
        case LFUN_INSET_FOOTNOTE:
-               code = InsetOld::FOOT_CODE;
+               code = InsetBase::FOOT_CODE;
                break;
        case LFUN_TABULAR_INSERT:
-               code = InsetOld::TABULAR_CODE;
+               code = InsetBase::TABULAR_CODE;
                break;
        case LFUN_INSET_MARGINAL:
-               code = InsetOld::MARGIN_CODE;
+               code = InsetBase::MARGIN_CODE;
                break;
        case LFUN_INSET_FLOAT:
        case LFUN_INSET_WIDE_FLOAT:
-               code = InsetOld::FLOAT_CODE;
+               code = InsetBase::FLOAT_CODE;
                break;
        case LFUN_INSET_WRAP:
-               code = InsetOld::WRAP_CODE;
+               code = InsetBase::WRAP_CODE;
                break;
        case LFUN_FLOAT_LIST:
-               code = InsetOld::FLOAT_LIST_CODE;
+               code = InsetBase::FLOAT_LIST_CODE;
                break;
 #if 0
        case LFUN_INSET_LIST:
-               code = InsetOld::LIST_CODE;
+               code = InsetBase::LIST_CODE;
                break;
        case LFUN_INSET_THEOREM:
-               code = InsetOld::THEOREM_CODE;
+               code = InsetBase::THEOREM_CODE;
                break;
 #endif
        case LFUN_INSET_CAPTION:
-               code = InsetOld::CAPTION_CODE;
+               code = InsetBase::CAPTION_CODE;
                break;
        case LFUN_INSERT_NOTE:
-               code = InsetOld::NOTE_CODE;
+               code = InsetBase::NOTE_CODE;
                break;
        case LFUN_INSERT_CHARSTYLE:
-               code = InsetOld::CHARSTYLE_CODE;
+               code = InsetBase::CHARSTYLE_CODE;
                if (buf->params().getLyXTextClass().charstyles().empty())
                        enable = false;
                break;
        case LFUN_INSERT_BOX:
-               code = InsetOld::BOX_CODE;
+               code = InsetBase::BOX_CODE;
                break;
        case LFUN_INSERT_BRANCH:
-               code = InsetOld::BRANCH_CODE;
+               code = InsetBase::BRANCH_CODE;
                if (buf->params().branchlist().empty())
                        enable = false;
                break;
        case LFUN_INSERT_LABEL:
-               code = InsetOld::LABEL_CODE;
+               code = InsetBase::LABEL_CODE;
                break;
        case LFUN_INSET_OPTARG:
-               code = InsetOld::OPTARG_CODE;
+               code = InsetBase::OPTARG_CODE;
                break;
        case LFUN_ENVIRONMENT_INSERT:
-               code = InsetOld::BOX_CODE;
+               code = InsetBase::BOX_CODE;
                break;
        case LFUN_INDEX_INSERT:
-               code = InsetOld::INDEX_CODE;
+               code = InsetBase::INDEX_CODE;
                break;
        case LFUN_INDEX_PRINT:
-               code = InsetOld::INDEX_PRINT_CODE;
+               code = InsetBase::INDEX_PRINT_CODE;
                break;
        case LFUN_TOC_INSERT:
-               code = InsetOld::TOC_CODE;
+               code = InsetBase::TOC_CODE;
                break;
        case LFUN_HTMLURL:
        case LFUN_URL:
-               code = InsetOld::URL_CODE;
+               code = InsetBase::URL_CODE;
                break;
        case LFUN_QUOTE:
                // always allow this, since we will inset a raw quote
@@ -1755,22 +1762,22 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
        case LFUN_MENU_SEPARATOR:
        case LFUN_LDOTS:
        case LFUN_END_OF_SENTENCE:
-               code = InsetOld::SPECIALCHAR_CODE;
+               code = InsetBase::SPECIALCHAR_CODE;
                break;
        case LFUN_SPACE_INSERT:
                // slight hack: we know this is allowed in math mode
                if (cur.inTexted())
-                       code = InsetOld::SPACE_CODE;
+                       code = InsetBase::SPACE_CODE;
                break;
        case LFUN_INSET_DIALOG_SHOW: {
                InsetBase * inset = cur.nextInset();
                enable = inset;
                if (inset) {
                        code = inset->lyxCode();
-                       if (!(code == InsetOld::INCLUDE_CODE
-                               || code == InsetOld::BIBTEX_CODE
-                               || code == InsetOld::FLOAT_LIST_CODE
-                               || code == InsetOld::TOC_CODE))
+                       if (!(code == InsetBase::INCLUDE_CODE
+                               || code == InsetBase::BIBTEX_CODE
+                               || code == InsetBase::FLOAT_LIST_CODE
+                               || code == InsetBase::TOC_CODE))
                                enable = false;
                }
                break;
@@ -1779,7 +1786,7 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
                break;
        }
 
-       if (code != InsetOld::NO_CODE
+       if (code != InsetBase::NO_CODE
                        && (cur.empty() || !cur.inset().insetAllowed(code)))
                enable = false;
 
@@ -1907,6 +1914,8 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
        case LFUN_INSERT_MATH:
        case LFUN_INSERT_MATRIX:
        case LFUN_MATH_DELIM:
+       case LFUN_SUBSCRIPT:
+       case LFUN_SUPERSCRIPT:
        case LFUN_DEFAULT:
        case LFUN_UNDERLINE:
        case LFUN_FONT_SIZE:
@@ -1939,18 +1948,16 @@ bool LyXText::getStatus(LCursor & cur, FuncRequest const & cmd,
        case LFUN_PARAGRAPH_APPLY:
        case LFUN_ESCAPE:
        case LFUN_KEYMAP_TOGGLE:
-               // these are handled in our dispatch()
-               enable = true;
-               break;
-
        case LFUN_ENDBUF:
        case LFUN_BEGINNINGBUF:
+       case LFUN_BEGINNINGBUFSEL:
+       case LFUN_ENDBUFSEL:
+               // these are handled in our dispatch()
                enable = true;
                break;
 
        default:
-               enable = false;
-               break;
+               return false;
        }
        flag.enabled(enable);
        return true;