]> git.lyx.org Git - lyx.git/blobdiff - src/Text3.cpp
Try even harder to obtain an instant preview
[lyx.git] / src / Text3.cpp
index fc8846ea5bb755cfc0d94b72da79aef9d2044c99..f80bde6554d38c8fe10b5f0c9aaaec32fbec7aa1 100644 (file)
@@ -47,6 +47,7 @@
 #include "TextMetrics.h"
 #include "WordLangTuple.h"
 
+#include "frontends/alert.h"
 #include "frontends/Application.h"
 #include "frontends/Clipboard.h"
 #include "frontends/Selection.h"
@@ -381,7 +382,7 @@ static void outline(OutlineOp mode, Cursor & cur)
                        pit_type const newpit = distance(bgn, dest);
                        pit_type const len = distance(start, finish);
                        pit_type const deletepit = pit + len;
-                       buf.undo().recordUndo(cur, ATOMIC_UNDO, newpit, deletepit - 1);
+                       buf.undo().recordUndo(cur, newpit, deletepit - 1);
                        pars.splice(dest, start, finish);
                        cur.pit() = newpit;
                        break;
@@ -401,7 +402,7 @@ static void outline(OutlineOp mode, Cursor & cur)
                        }
                        // One such was found:
                        pit_type newpit = distance(bgn, dest);
-                       buf.undo().recordUndo(cur, ATOMIC_UNDO, pit, newpit - 1);
+                       buf.undo().recordUndo(cur, pit, newpit - 1);
                        pit_type const len = distance(start, finish);
                        pars.splice(dest, start, finish);
                        cur.pit() = newpit - len;
@@ -410,7 +411,7 @@ static void outline(OutlineOp mode, Cursor & cur)
                case OutlineIn:
                case OutlineOut: {
                        pit_type const len = distance(start, finish);
-                       buf.undo().recordUndo(cur, ATOMIC_UNDO, pit, pit + len - 1);
+                       buf.undo().recordUndo(cur, pit, pit + len - 1);
                        for (; start != finish; ++start) {
                                toclevel = buf.text().getTocLevel(distance(bgn, start));
                                if (toclevel == Layout::NOT_IN_TOC)
@@ -476,6 +477,8 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
        cur.noScreenUpdate();
 
        LBUFERR(this == cur.text());
+       
+       // NOTE: This should NOT be a reference. See commit 94a5481a.
        CursorSlice const oldTopSlice = cur.top();
        bool const oldBoundary = cur.boundary();
        bool const oldSelection = cur.selection();
@@ -494,7 +497,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 
        case LFUN_PARAGRAPH_MOVE_DOWN: {
                pit_type const pit = cur.pit();
-               recUndo(cur, pit, pit + 1);
+               cur.recordUndo(pit, pit + 1);
                cur.finishUndo();
                pars_.swap(pit, pit + 1);
                needsUpdate = true;
@@ -505,7 +508,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 
        case LFUN_PARAGRAPH_MOVE_UP: {
                pit_type const pit = cur.pit();
-               recUndo(cur, pit - 1, pit);
+               cur.recordUndo(pit - 1, pit);
                cur.finishUndo();
                pars_.swap(pit, pit - 1);
                --cur.pit();
@@ -524,7 +527,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                // FIXME: this don't work for multipart document!
                for (pit_type tmp = 0, end = pars_.size(); tmp != end; ++tmp) {
                        if (pars_[tmp].params().startOfAppendix()) {
-                               recUndo(cur, tmp);
+                               cur.recordUndo(tmp, tmp);
                                pars_[tmp].params().startOfAppendix(false);
                                break;
                        }
@@ -603,13 +606,14 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_CHAR_FORWARD:
-       case LFUN_CHAR_FORWARD_SELECT:
+       case LFUN_CHAR_FORWARD_SELECT: {
                //LYXERR0(" LFUN_CHAR_FORWARD[SEL]:\n" << cur);
                needsUpdate |= cur.selHandle(act == LFUN_CHAR_FORWARD_SELECT);
-               needsUpdate |= cursorForward(cur);
+               bool const cur_moved = cursorForward(cur);
+               needsUpdate |= cur_moved;
 
-               if (!needsUpdate && oldTopSlice == cur.top()
-                               && cur.boundary() == oldBoundary) {
+               if (!cur_moved && oldTopSlice == cur.top()
+                              && cur.boundary() == oldBoundary) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_FORWARD);
 
@@ -628,15 +632,17 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        }
                }
                break;
+       }
 
        case LFUN_CHAR_BACKWARD:
-       case LFUN_CHAR_BACKWARD_SELECT:
+       case LFUN_CHAR_BACKWARD_SELECT: {
                //lyxerr << "handle LFUN_CHAR_BACKWARD[_SELECT]:\n" << cur << endl;
                needsUpdate |= cur.selHandle(act == LFUN_CHAR_BACKWARD_SELECT);
-               needsUpdate |= cursorBackward(cur);
+               bool const cur_moved = cursorBackward(cur);
+               needsUpdate |= cur_moved;
 
-               if (!needsUpdate && oldTopSlice == cur.top()
-                       && cur.boundary() == oldBoundary) {
+               if (!cur_moved && oldTopSlice == cur.top()
+                              && cur.boundary() == oldBoundary) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_BACKWARD);
 
@@ -655,14 +661,16 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        }
                }
                break;
+       }
 
        case LFUN_CHAR_LEFT:
        case LFUN_CHAR_LEFT_SELECT:
                if (lyxrc.visual_cursor) {
                        needsUpdate |= cur.selHandle(act == LFUN_CHAR_LEFT_SELECT);
-                       needsUpdate |= cursorVisLeft(cur);
-                       if (!needsUpdate && oldTopSlice == cur.top()
-                                       && cur.boundary() == oldBoundary) {
+                       bool const cur_moved = cursorVisLeft(cur);
+                       needsUpdate |= cur_moved;
+                       if (!cur_moved && oldTopSlice == cur.top()
+                                      && cur.boundary() == oldBoundary) {
                                cur.undispatched();
                                cmd = FuncRequest(LFUN_FINISHED_LEFT);
                        }
@@ -683,9 +691,10 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_CHAR_RIGHT_SELECT:
                if (lyxrc.visual_cursor) {
                        needsUpdate |= cur.selHandle(cmd.action() == LFUN_CHAR_RIGHT_SELECT);
-                       needsUpdate |= cursorVisRight(cur);
-                       if (!needsUpdate && oldTopSlice == cur.top()
-                                       && cur.boundary() == oldBoundary) {
+                       bool const cur_moved = cursorVisRight(cur);
+                       needsUpdate |= cur_moved;
+                       if (!cur_moved && oldTopSlice == cur.top()
+                                      && cur.boundary() == oldBoundary) {
                                cur.undispatched();
                                cmd = FuncRequest(LFUN_FINISHED_RIGHT);
                        }
@@ -794,9 +803,10 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_WORD_RIGHT_SELECT:
                if (lyxrc.visual_cursor) {
                        needsUpdate |= cur.selHandle(cmd.action() == LFUN_WORD_RIGHT_SELECT);
-                       needsUpdate |= cursorVisRightOneWord(cur);
-                       if (!needsUpdate && oldTopSlice == cur.top()
-                                       && cur.boundary() == oldBoundary) {
+                       bool const cur_moved = cursorVisRightOneWord(cur);
+                       needsUpdate |= cur_moved;
+                       if (!cur_moved && oldTopSlice == cur.top()
+                                      && cur.boundary() == oldBoundary) {
                                cur.undispatched();
                                cmd = FuncRequest(LFUN_FINISHED_RIGHT);
                        }
@@ -814,12 +824,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_WORD_FORWARD:
-       case LFUN_WORD_FORWARD_SELECT:
+       case LFUN_WORD_FORWARD_SELECT: {
                needsUpdate |= cur.selHandle(cmd.action() == LFUN_WORD_FORWARD_SELECT);
-               needsUpdate |= cursorForwardOneWord(cur);
+               bool const cur_moved = cursorForwardOneWord(cur);
+               needsUpdate |= cur_moved;
 
-               if (!needsUpdate && oldTopSlice == cur.top()
-                               && cur.boundary() == oldBoundary) {
+               if (!cur_moved && oldTopSlice == cur.top()
+                              && cur.boundary() == oldBoundary) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_FORWARD);
 
@@ -838,14 +849,16 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        }
                }
                break;
+       }
 
        case LFUN_WORD_LEFT:
        case LFUN_WORD_LEFT_SELECT:
                if (lyxrc.visual_cursor) {
                        needsUpdate |= cur.selHandle(cmd.action() == LFUN_WORD_LEFT_SELECT);
-                       needsUpdate |= cursorVisLeftOneWord(cur);
-                       if (!needsUpdate && oldTopSlice == cur.top()
-                                       && cur.boundary() == oldBoundary) {
+                       bool const cur_moved = cursorVisLeftOneWord(cur);
+                       needsUpdate |= cur_moved;
+                       if (!cur_moved && oldTopSlice == cur.top()
+                                      && cur.boundary() == oldBoundary) {
                                cur.undispatched();
                                cmd = FuncRequest(LFUN_FINISHED_LEFT);
                        }
@@ -863,12 +876,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_WORD_BACKWARD:
-       case LFUN_WORD_BACKWARD_SELECT:
+       case LFUN_WORD_BACKWARD_SELECT: {
                needsUpdate |= cur.selHandle(cmd.action() == LFUN_WORD_BACKWARD_SELECT);
-               needsUpdate |= cursorBackwardOneWord(cur);
+               bool const cur_moved = cursorBackwardOneWord(cur);
+               needsUpdate |= cur_moved;
 
-               if (!needsUpdate && oldTopSlice == cur.top()
-                               && cur.boundary() == oldBoundary) {
+               if (!cur_moved && oldTopSlice == cur.top()
+                              && cur.boundary() == oldBoundary) {
                        cur.undispatched();
                        cmd = FuncRequest(LFUN_FINISHED_BACKWARD);
 
@@ -888,6 +902,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        }
                }
                break;
+       }
 
        case LFUN_WORD_SELECT: {
                selectWord(cur, WHOLE_WORD);
@@ -1101,13 +1116,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        /*
                        Paragraph & par = pars_[cur.pit()];
                        if (inset->lyxCode() == LABEL_CODE
-                               && !par.layout().counter.empty() {
+                               && !par.layout().counter.empty()) {
                                // Go to the end of the paragraph
                                // Warning: Because of Change-Tracking, the last
                                // position is 'size()' and not 'size()-1':
                                cur.pos() = par.size();
                                // Insert a new paragraph
-                               FuncRequest fr(LFUN_BREAK_PARAGRAPH);
+                               FuncRequest fr(LFUN_PARAGRAPH_BREAK);
                                dispatch(cur, fr);
                        }
                        */
@@ -1186,6 +1201,14 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        specialChar(cur, InsetSpecialChar::END_OF_SENTENCE);
                else if (name == "menu-separator")
                        specialChar(cur, InsetSpecialChar::MENU_SEPARATOR);
+               else if (name == "lyx")
+                       specialChar(cur, InsetSpecialChar::PHRASE_LYX);
+               else if (name == "tex")
+                       specialChar(cur, InsetSpecialChar::PHRASE_TEX);
+               else if (name == "latex")
+                       specialChar(cur, InsetSpecialChar::PHRASE_LATEX);
+               else if (name == "latex2e")
+                       specialChar(cur, InsetSpecialChar::PHRASE_LATEX2E);
                else if (name.empty())
                        lyxerr << "LyX function 'specialchar-insert' needs an argument." << endl;
                else
@@ -1279,8 +1302,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                        else if (arg == "wmf")
                                type = Clipboard::WmfGraphicsType;
                        else
-                               // We used to assert, but couldn't the argument come from, say, the
-                               // minibuffer and just be mistyped?
+                               // we also check in getStatus()
                                LYXERR0("Unrecognized graphics type: " << arg);
 
                        pasteClipboardGraphics(cur, bv->buffer().errorList("Paste"), type);
@@ -1289,6 +1311,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                bv->buffer().errors("Paste");
                cur.clearSelection(); // bug 393
                cur.finishUndo();
+               bv->buffer().updatePreviews();
                break;
        }
 
@@ -2264,6 +2287,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                                arg += " lang=" + from_ascii(cur.getFont().language()->lang());
                        }
                }
+               if (lyxrc.thesaurusdir_path.empty()) {
+                       frontend::Alert::warning(_("Path to thesaurus directory not set!"),
+                                       _("The path to the thesaurus directory has not been specified.\n"
+                                         "The thesaurus is not functional.\n"
+                                         "Please refer to sec. 6.15.1 of the User's Guide for setup\n"
+                                         "instructions."));
+               }
                bv->showDialog("thesaurus", to_utf8(arg));
                break;
        }
@@ -2280,8 +2310,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                                word = cur.selectionAsString(false);
                        }
                        lang = const_cast<Language *>(cur.getFont().language());
-               } else
+               } else if (cmd.getArg(1).empty()) {
+                       // optional language argument is missing
+                       // use the language at cursor position
+                       lang = const_cast<Language *>(cur.getFont().language());
+               } else {
                        lang = const_cast<Language *>(languages.getLanguage(cmd.getArg(1)));
+               }
                WordLangTuple wl(word, lang);
                theSpellChecker()->insert(wl);
                break;
@@ -2299,8 +2334,11 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                                word = cur.selectionAsString(false);
                        }
                        lang = const_cast<Language *>(cur.getFont().language());
-               } else
+               } else if (cmd.getArg(1).empty()) {
+                       lang = const_cast<Language *>(cur.getFont().language());
+               } else {
                        lang = const_cast<Language *>(languages.getLanguage(cmd.getArg(1)));
+               }
                WordLangTuple wl(word, lang);
                theSpellChecker()->accept(wl);
                break;
@@ -2318,8 +2356,11 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
                                word = cur.selectionAsString(false);
                        }
                        lang = const_cast<Language *>(cur.getFont().language());
-               } else
+               } else if (cmd.getArg(1).empty()) {
+                       lang = const_cast<Language *>(cur.getFont().language());
+               } else {
                        lang = const_cast<Language *>(languages.getLanguage(cmd.getArg(1)));
+               }
                WordLangTuple wl(word, lang);
                theSpellChecker()->remove(wl);
                break;
@@ -2902,6 +2943,9 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
                }
 
                // unknown argument
+               LYXERR0("Unrecognized graphics type: " << arg);
+               // we don't want to assert if the user just mistyped the LFUN
+               LATTEST(cmd.origin() != FuncRequest::INTERNAL);
                enable = false;
                break;
         }
@@ -2995,18 +3039,31 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
                break;
 
        case LFUN_PARAGRAPH_BREAK:
-               enable = cur.inset().getLayout().isMultiPar();
+               enable = inset().allowMultiPar();
                break;
 
        case LFUN_SPELLING_ADD:
        case LFUN_SPELLING_IGNORE:
        case LFUN_SPELLING_REMOVE:
-               enable = theSpellChecker();
+               enable = theSpellChecker() != NULL;
+               if (enable && !cmd.getArg(1).empty()) {
+                       // validate explicitly given language
+                       Language const * const lang = const_cast<Language *>(languages.getLanguage(cmd.getArg(1)));
+                       enable &= lang != NULL;
+               }
                break;
 
-       case LFUN_LAYOUT:
+       case LFUN_LAYOUT: {
                enable = !cur.inset().forcePlainLayout();
+
+               docstring layout = cmd.argument();
+               if (layout.empty()) {
+                       DocumentClass const & tclass = cur.buffer()->params().documentClass();
+                       layout = tclass.defaultLayoutName();
+               }
+               flag.setOnOff(layout == cur.paragraph().layout().name());
                break;
+       }
 
        case LFUN_ENVIRONMENT_SPLIT: {
                if (cmd.argument() == "outer") {