]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView_pimpl.C
"Inter-word Space"
[lyx.git] / src / BufferView_pimpl.C
index 696f9197c14093ed3baec1bc7fa8a8cd03040c0b..d167010f8ed9f39a8cbb46a3863c64f421d6eff5 100644 (file)
@@ -8,61 +8,53 @@
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "BufferView_pimpl.h"
-#include "frontends/WorkArea.h"
-#include "frontends/screen.h"
-#include "frontends/LyXScreenFactory.h"
-#include "frontends/WorkAreaFactory.h"
-#include "frontends/Dialogs.h"
-#include "frontends/Alert.h"
-#include "frontends/FileDialog.h"
-#include "frontends/mouse_state.h"
-#include "lyxtext.h"
-#include "lyxrow.h"
-#include "paragraph.h"
-#include "frontends/LyXView.h"
-#include "commandtags.h"
-#include "lyxfunc.h"
-#include "debug.h"
-#include "bufferview_funcs.h"
-#include "TextCache.h"
 #include "bufferlist.h"
-#include "lyxrc.h"
-#include "intl.h"
-// added for Dispatch functions
-#include "lyx_cb.h"
-#include "lyx_main.h"
+#include "buffer.h"
+#include "bufferview_funcs.h"
+#include "lfuns.h"
+#include "debug.h"
+#include "factory.h"
 #include "FloatList.h"
+#include "funcrequest.h"
 #include "gettext.h"
+#include "intl.h"
+#include "iterators.h"
+#include "Lsstream.h"
+#include "lyx_cb.h" // added for Dispatch functions
+#include "lyx_main.h"
+#include "lyxfind.h"
+#include "lyxfunc.h"
+#include "lyxtext.h"
+#include "lyxrc.h"
+#include "lyxrow.h"
+#include "paragraph.h"
 #include "ParagraphParameters.h"
+#include "TextCache.h"
 #include "undo_funcs.h"
-#include "funcrequest.h"
 
-#include "insets/insetbib.h"
-#include "insets/insettext.h"
-#include "insets/inseturl.h"
-#include "insets/insetlatexaccent.h"
-#include "insets/insettoc.h"
-#include "insets/insetref.h"
-#include "insets/insetparent.h"
-#include "insets/insetindex.h"
-#include "insets/insetinclude.h"
-#include "insets/insetcite.h"
-#include "insets/insetgraphics.h"
-#include "insets/insetmarginal.h"
-#include "insets/insetcaption.h"
 #include "insets/insetfloatlist.h"
+#include "insets/insetgraphics.h"
+#include "insets/insetinclude.h"
+#include "insets/insetref.h"
+#include "insets/insettext.h"
+
+#include "frontends/Alert.h"
+#include "frontends/Dialogs.h"
+#include "frontends/FileDialog.h"
+#include "frontends/LyXView.h"
+#include "frontends/LyXScreenFactory.h"
+#include "frontends/mouse_state.h"
+#include "frontends/screen.h"
+#include "frontends/WorkArea.h"
+#include "frontends/WorkAreaFactory.h"
 
 #include "mathed/formulabase.h"
 
 #include "graphics/Previews.h"
 
 #include "support/LAssert.h"
-#include "support/lstrings.h"
+#include "support/tostr.h"
 #include "support/filetools.h"
 
 #include <boost/bind.hpp>
@@ -81,6 +73,7 @@ using std::make_pair;
 using std::min;
 
 using lyx::pos_type;
+using namespace bv_funcs;
 
 extern BufferList bufferlist;
 
@@ -120,7 +113,7 @@ BufferView::Pimpl::Pimpl(BufferView * bv, LyXView * owner,
        resizecon = workarea().workAreaResize
                .connect(boost::bind(&BufferView::Pimpl::workAreaResize, this));
        dispatchcon = workarea().dispatch
-               .connect(boost::bind(&BufferView::Pimpl::dispatch, this, _1));
+               .connect(boost::bind(&BufferView::Pimpl::workAreaDispatch, this, _1));
        kpresscon = workarea().workAreaKeyPress
                .connect(boost::bind(&BufferView::Pimpl::workAreaKeyPress, this, _1, _2));
        selectioncon = workarea().selectionRequested
@@ -156,7 +149,7 @@ Painter & BufferView::Pimpl::painter() const
 void BufferView::Pimpl::buffer(Buffer * b)
 {
        lyxerr[Debug::INFO] << "Setting buffer in BufferView ("
-                           << b << ")" << endl;
+                           << b << ')' << endl;
        if (buffer_) {
                buffer_->delUser(bv_);
 
@@ -174,7 +167,9 @@ void BufferView::Pimpl::buffer(Buffer * b)
        // set current buffer
        buffer_ = b;
 
-       if (bufferlist.getState() == BufferList::CLOSING) return;
+       // if we're quitting lyx, don't bother updating stuff
+       if (quitting)
+               return;
 
        // if we are closing the buffer, use the first buffer as current
        if (!buffer_) {
@@ -191,10 +186,9 @@ void BufferView::Pimpl::buffer(Buffer * b)
                }
 
                // FIXME: needed when ?
-               bv_->text->first_y =
-                       screen().topCursorVisible(bv_->text->cursor, bv_->text->first_y);
+               bv_->text->top_y(screen().topCursorVisible(bv_->text->cursor, bv_->text->top_y()));
 
-               // Similarly, buffer-dependent dialogs should be updated or
+               // Buffer-dependent dialogs should be updated or
                // hidden. This should go here because some dialogs (eg ToC)
                // require bv_->text.
                owner_->getDialogs().updateBufferDependent(true);
@@ -217,6 +211,13 @@ void BufferView::Pimpl::buffer(Buffer * b)
        owner_->updateLayoutChoice();
        owner_->updateWindowTitle();
 
+       if (buffer_) {
+               // Don't forget to update the Layout
+               string const layoutname =
+                       bv_->text->cursor.par()->layout()->name();
+               owner_->setLayout(layoutname);
+       }
+
        if (grfx::Previews::activated() && buffer_)
                grfx::Previews::get().generateBufferPreviews(*buffer_);
 }
@@ -233,9 +234,12 @@ bool BufferView::Pimpl::fitCursor()
                ret = screen().fitCursor(bv_->text, bv_);
        }
 
-       bv_->owner()->getDialogs().updateParagraph();
-       if (ret)
-               updateScrollbar();
+       dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
+
+       // We need to always update, in case we did a
+       // paste and we stayed anchored to a row, but
+       // the actual height of the doc changed ...
+       updateScrollbar();
        return ret;
 }
 
@@ -256,9 +260,9 @@ int BufferView::Pimpl::resizeCurrentBuffer()
 {
        lyxerr[Debug::INFO] << "resizeCurrentBuffer" << endl;
 
-       Paragraph * par = 0;
-       Paragraph * selstartpar = 0;
-       Paragraph * selendpar = 0;
+       ParagraphList::iterator par;
+       ParagraphList::iterator selstartpar;
+       ParagraphList::iterator selendpar;
        UpdatableInset * the_locking_inset = 0;
 
        pos_type pos = 0;
@@ -267,7 +271,7 @@ int BufferView::Pimpl::resizeCurrentBuffer()
        bool selection = false;
        bool mark_set  = false;
 
-       owner_->prohibitInput();
+       owner_->busy(true);
 
        owner_->message(_("Formatting document..."));
 
@@ -300,26 +304,32 @@ int BufferView::Pimpl::resizeCurrentBuffer()
                        //      bv_->text->owner(bv_);
                        if (lyxerr.debugging())
                                textcache.show(lyxerr, "resizeCurrentBuffer");
+
+                       buffer_->resizeInsets(bv_);
                } else {
                        bv_->text = new LyXText(bv_);
                        bv_->text->init(bv_);
                        //buffer_->resizeInsets(bv_);
                }
+
+               par = bv_->text->ownerParagraphs().end();
+               selstartpar = bv_->text->ownerParagraphs().end();
+               selendpar = bv_->text->ownerParagraphs().end();
        }
 
-       if (par) {
+       if (par != bv_->text->ownerParagraphs().end()) {
                bv_->text->selection.set(true);
                // At this point just to avoid the Delete-Empty-Paragraph-
                // Mechanism when setting the cursor.
                bv_->text->selection.mark(mark_set);
                if (selection) {
-                       bv_->text->setCursor(bv_, selstartpar, selstartpos);
+                       bv_->text->setCursor(selstartpar, selstartpos);
                        bv_->text->selection.cursor = bv_->text->cursor;
-                       bv_->text->setCursor(bv_, selendpar, selendpos);
-                       bv_->text->setSelection(bv_);
-                       bv_->text->setCursor(bv_, par, pos);
+                       bv_->text->setCursor(selendpar, selendpos);
+                       bv_->text->setSelection();
+                       bv_->text->setCursor(par, pos);
                } else {
-                       bv_->text->setCursor(bv_, par, pos);
+                       bv_->text->setCursor(par, pos);
                        bv_->text->selection.cursor = bv_->text->cursor;
                        bv_->text->selection.set(false);
                }
@@ -327,10 +337,13 @@ int BufferView::Pimpl::resizeCurrentBuffer()
                bv_->theLockingInset(the_locking_inset);
        }
 
-       bv_->text->first_y = screen().topCursorVisible(bv_->text->cursor, bv_->text->first_y);
+       bv_->text->top_y(screen().topCursorVisible(bv_->text->cursor, bv_->text->top_y()));
 
        switchKeyMap();
-       owner_->allowInput();
+       owner_->busy(false);
+
+       // reset the "Formatting..." message
+       owner_->clearMessage();
 
        updateScrollbar();
 
@@ -355,10 +368,10 @@ void BufferView::Pimpl::updateScrollbar()
 
        LyXText const & t = *bv_->text;
 
-       lyxerr[Debug::GUI] << "Updating scrollbar: h " << t.height << ", first_y "
-               << t.first_y << ", default height " << t.defaultHeight() << endl;
+       lyxerr[Debug::GUI] << "Updating scrollbar: h " << t.height << ", top_y() "
+               << t.top_y() << ", default height " << defaultRowHeight() << endl;
 
-       workarea().setScrollbarParams(t.height, t.first_y, t.defaultHeight());
+       workarea().setScrollbarParams(t.height, t.top_y(), defaultRowHeight());
 }
 
 
@@ -366,40 +379,51 @@ void BufferView::Pimpl::scrollDocView(int value)
 {
        lyxerr[Debug::GUI] << "scrollDocView of " << value << endl;
 
-       if (!buffer_) return;
+       if (!buffer_)
+               return;
+
+       screen().hideCursor();
 
        screen().draw(bv_->text, bv_, value);
 
-       if (!lyxrc.cursor_follows_scrollbar) {
+       if (!lyxrc.cursor_follows_scrollbar)
                return;
-       }
 
        LyXText * vbt = bv_->text;
 
-       int const height = vbt->defaultHeight();
-       int const first = static_cast<int>((bv_->text->first_y + height));
-       int const last = static_cast<int>((bv_->text->first_y + workarea().workHeight() - height));
+       int const height = defaultRowHeight();
+       int const first = static_cast<int>((bv_->text->top_y() + height));
+       int const last = static_cast<int>((bv_->text->top_y() + workarea().workHeight() - height));
 
        if (vbt->cursor.y() < first)
-               vbt->setCursorFromCoordinates(bv_, 0, first);
+               vbt->setCursorFromCoordinates(0, first);
        else if (vbt->cursor.y() > last)
-               vbt->setCursorFromCoordinates(bv_, 0, last);
+               vbt->setCursorFromCoordinates(0, last);
+
+       owner_->updateLayoutChoice();
 }
 
 
-int BufferView::Pimpl::scroll(long time)
+void BufferView::Pimpl::scroll(int lines)
 {
-       if (!buffer_)
-               return 0;
+       if (!buffer_) {
+               return;
+       }
 
        LyXText const * t = bv_->text;
+       int const line_height = defaultRowHeight();
 
-       double const diff = t->defaultHeight()
-               + double(time) * double(time) * 0.125;
+       // The new absolute coordinate
+       int new_top_y = t->top_y() + lines * line_height;
 
-       scrollDocView(int(diff));
-       workarea().setScrollbarParams(t->height, t->first_y, t->defaultHeight());
-       return 0;
+       // Restrict to a valid value
+       new_top_y = std::min(t->height - 4 * line_height, new_top_y);
+       new_top_y = std::max(0, new_top_y);
+
+       scrollDocView(new_top_y);
+
+       // Update the scrollbar.
+       workarea().setScrollbarParams(t->height, t->top_y(), defaultRowHeight());
 }
 
 
@@ -407,6 +431,18 @@ void BufferView::Pimpl::workAreaKeyPress(LyXKeySymPtr key,
                                         key_modifier::state state)
 {
        bv_->owner()->getLyXFunc().processKeySym(key, state);
+
+       /* This is perhaps a bit of a hack. When we move
+        * around, or type, it's nice to be able to see
+        * the cursor immediately after the keypress. So
+        * we reset the toggle timeout and force the visibility
+        * of the cursor. Note we cannot do this inside
+        * dispatch() itself, because that's called recursively.
+        */
+       if (available()) {
+               cursor_timeout.restart();
+               screen().showCursor(*bv_);
+       }
 }
 
 
@@ -439,10 +475,9 @@ void BufferView::Pimpl::selectionRequested()
 void BufferView::Pimpl::selectionLost()
 {
        if (available()) {
-               hideCursor();
+               screen().hideCursor();
                toggleSelection();
                bv_->getLyXText()->clearSelection();
-               showCursor();
                bv_->text->xsel_cache.set(false);
        }
 }
@@ -498,95 +533,44 @@ void BufferView::Pimpl::workAreaResize()
 void BufferView::Pimpl::update()
 {
        if (!bv_->theLockingInset() || !bv_->theLockingInset()->nodraw()) {
-               LyXText::text_status st = bv_->text->status();
-               screen().update(bv_->text, bv_);
-               bool fitc = false;
-               while (bv_->text->status() == LyXText::CHANGED_IN_DRAW) {
-                       bv_->text->fullRebreak(bv_);
-                       st = LyXText::NEED_MORE_REFRESH;
-                       bv_->text->setCursor(bv_, bv_->text->cursor.par(),
-                                            bv_->text->cursor.pos());
-                       if (bv_->text->selection.set()) {
-                               bv_->text->setCursor(bv_, bv_->text->selection.start,
-                                                    bv_->text->selection.start.par(),
-                                                    bv_->text->selection.start.pos());
-                               bv_->text->setCursor(bv_, bv_->text->selection.end,
-                                                    bv_->text->selection.end.par(),
-                                                    bv_->text->selection.end.pos());
-                       }
-                       fitc = true;
-                       bv_->text->status(bv_, st);
-                       screen().update(bv_->text, bv_);
-               }
-               // do this here instead of in the screen::update because of
-               // the above loop!
-               bv_->text->status(bv_, LyXText::UNCHANGED);
-               if (fitc)
-                       fitCursor();
+               screen().update(*bv_);
+               bv_->text->clearPaint();
        }
 }
 
-// Values used when calling update:
-// -3 - update
-// -2 - update, move sel_cursor if selection, fitcursor
-// -1 - update, move sel_cursor if selection, fitcursor, mark dirty
-//  0 - update, move sel_cursor if selection, fitcursor
-//  1 - update, move sel_cursor if selection, fitcursor, mark dirty
-//  3 - update, move sel_cursor if selection
-//
-// update -
-// a simple redraw of the parts that need refresh
-//
-// move sel_cursor if selection -
-// the text's sel_cursor is moved if there is selection is progress
-//
-// fitcursor -
-// fitCursor() is called and the scrollbar updated
-//
-// mark dirty -
-// the buffer is marked dirty.
-//
-// enum {
-//       UPDATE = 0,
-//       SELECT = 1,
-//       FITCUR = 2,
-//       CHANGE = 4
-// };
-//
-// UPDATE_ONLY = UPDATE;
-// UPDATE_SELECT = UPDATE | SELECT;
-// UPDATE_SELECT_MOVE = UPDATE | SELECT | FITCUR;
-// UPDATE_SELECT_MOVE_AFTER_CHANGE = UPDATE | SELECT | FITCUR | CHANGE;
-//
-// update(-3) -> update(0)         -> update(0) -> update(UPDATE)
-// update(-2) -> update(1 + 2)     -> update(3) -> update(SELECT|FITCUR)
-// update(-1) -> update(1 + 2 + 4) -> update(7) -> update(SELECT|FITCUR|CHANGE)
-// update(1)  -> update(1 + 2 + 4) -> update(7) -> update(SELECT|FITCUR|CHANGE)
-// update(3)  -> update(1)         -> update(1) -> update(SELECT)
 
 void BufferView::Pimpl::update(LyXText * text, BufferView::UpdateCodes f)
 {
-       owner_->updateLayoutChoice();
-
        if (!text->selection.set() && (f & SELECT)) {
                text->selection.cursor = text->cursor;
        }
 
-       text->fullRebreak(bv_);
+       text->fullRebreak();
 
        if (text->inset_owner) {
                text->inset_owner->setUpdateStatus(bv_, InsetText::NONE);
-               updateInset(text->inset_owner, false);
+               updateInset(text->inset_owner);
        } else {
                update();
        }
+}
+
+
+void BufferView::Pimpl::update(BufferView::UpdateCodes f)
+{
+       LyXText * text = bv_->text;
 
-       if ((f & FITCUR)) {
-               fitCursor();
+       if (!text->selection.set() && (f & SELECT)) {
+               text->selection.cursor = text->cursor;
        }
 
-       if ((f & CHANGE)) {
-               buffer_->markDirty();
+       text->fullRebreak();
+
+       if (text->inset_owner) {
+               text->inset_owner->setUpdateStatus(bv_, InsetText::NONE);
+               updateInset(text->inset_owner);
+       } else {
+               update();
        }
 }
 
@@ -599,15 +583,7 @@ void BufferView::Pimpl::cursorToggle()
                return;
        }
 
-       /* FIXME */
-       extern void reapSpellchecker(void);
-       reapSpellchecker();
-
-       if (!bv_->theLockingInset()) {
-               screen().cursorToggle(bv_);
-       } else {
-               bv_->theLockingInset()->toggleInsetCursor(bv_);
-       }
+       screen().toggleCursor(*bv_);
 
        cursor_timeout.restart();
 }
@@ -621,6 +597,21 @@ bool BufferView::Pimpl::available() const
 }
 
 
+Change const BufferView::Pimpl::getCurrentChange()
+{
+       if (!bv_->buffer()->params.tracking_changes)
+               return Change(Change::UNCHANGED);
+
+       LyXText * t(bv_->getLyXText());
+
+       if (!t->selection.set())
+               return Change(Change::UNCHANGED);
+
+       LyXCursor const & cur(t->selection.start);
+       return cur.par()->lookupChangeFull(cur.pos());
+}
+
+
 void BufferView::Pimpl::beforeChange(LyXText * text)
 {
        toggleSelection();
@@ -635,11 +626,8 @@ void BufferView::Pimpl::savePosition(unsigned int i)
        saved_positions[i] = Position(buffer_->fileName(),
                                      bv_->text->cursor.par()->id(),
                                      bv_->text->cursor.pos());
-       if (i > 0) {
-               ostringstream str;
-               str << _("Saved bookmark") << ' ' << i;
-               owner_->message(str.str().c_str());
-       }
+       if (i > 0)
+               owner_->message(bformat(_("Saved bookmark %1$s"), tostr(i)));
 }
 
 
@@ -659,19 +647,16 @@ void BufferView::Pimpl::restorePosition(unsigned int i)
                if (b != 0) buffer(b);
        }
 
-       Paragraph * par = buffer_->getParFromID(saved_positions[i].par_id);
-       if (!par)
+       ParIterator par = buffer_->getParFromID(saved_positions[i].par_id);
+       if (par == buffer_->par_iterator_end())
                return;
 
-       bv_->text->setCursor(bv_, par,
-                            min(par->size(), saved_positions[i].par_pos));
+       bv_->text->setCursor(*par,
+                            min((*par)->size(), saved_positions[i].par_pos));
 
-       update(bv_->text, BufferView::SELECT | BufferView::FITCUR);
-       if (i > 0) {
-               ostringstream str;
-               str << _("Moved to bookmark") << ' ' << i;
-               owner_->message(str.str().c_str());
-       }
+       update(BufferView::SELECT);
+       if (i > 0)
+               owner_->message(bformat(_("Moved to bookmark %1$s"), tostr(i)));
 }
 
 
@@ -713,22 +698,6 @@ void BufferView::Pimpl::insetUnlock()
 }
 
 
-void BufferView::Pimpl::showCursor()
-{
-       if (bv_->theLockingInset())
-               bv_->theLockingInset()->showInsetCursor(bv_);
-       else
-               screen().showCursor(bv_->text, bv_);
-}
-
-
-void BufferView::Pimpl::hideCursor()
-{
-       if (!bv_->theLockingInset())
-               screen().hideCursor();
-}
-
-
 void BufferView::Pimpl::toggleSelection(bool b)
 {
        if (bv_->theLockingInset())
@@ -755,18 +724,20 @@ void BufferView::Pimpl::center()
                new_y = t->cursor.y() - half_height;
        }
 
+       // FIXME: look at this comment again ...
+
        // FIXME: can we do this w/o calling screen directly ?
-       // This updates first_y but means the fitCursor() call
+       // This updates top_y() but means the fitCursor() call
        // from the update(FITCUR) doesn't realise that we might
        // have moved (e.g. from GOTOPARAGRAPH), so doesn't cause
        // the scrollbar to be updated as it should, so we have
        // to do it manually. Any operation that does a center()
-       // and also might have moved first_y must make sure to call
+       // and also might have moved top_y() must make sure to call
        // updateScrollbar() currently. Never mind that this is a
-       // pretty obfuscated way of updating t->first_y
+       // pretty obfuscated way of updating t->top_y()
        screen().draw(t, bv_, new_y);
 
-       update(t, BufferView::SELECT | BufferView::FITCUR);
+       update(BufferView::SELECT);
 }
 
 
@@ -846,8 +817,7 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
                                initpath = trypath;
                }
 
-               FileDialog fileDlg(bv_->owner(),
-                                  _("Select LyX document to insert"),
+               FileDialog fileDlg(_("Select LyX document to insert"),
                        LFUN_FILE_INSERT,
                        make_pair(string(_("Documents|#o#O")),
                                  string(lyxrc.document_path)),
@@ -855,7 +825,7 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
                                  string(AddPath(system_lyxdir, "examples"))));
 
                FileDialog::Result result =
-                       fileDlg.Select(initpath,
+                       fileDlg.open(initpath,
                                       _("*.lyx| LyX Documents (*.lyx)"));
 
                if (result.first == FileDialog::Later)
@@ -874,91 +844,107 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
        // necessary
        filename = FileSearch(string(), filename, "lyx");
 
-       string const disp_fn(MakeDisplayPath(filename));
-
-       ostringstream s1;
-       s1 << _("Inserting document") << ' '
-          << disp_fn << " ...";
-       owner_->message(s1.str().c_str());
+       string const disp_fn = MakeDisplayPath(filename);
+       owner_->message(bformat(_("Inserting document %1$s..."), disp_fn));
        bool const res = bv_->insertLyXFile(filename);
-       if (res) {
-               ostringstream str;
-               str << _("Document") << ' ' << disp_fn
-                   << ' ' << _("inserted.");
-               owner_->message(str.str().c_str());
+       if (res)
+               owner_->message(bformat(_("Document %1$s inserted."), disp_fn));
+       else
+               owner_->message(bformat(_("Could not insert document %1$s"), disp_fn));
+}
+
+
+void BufferView::Pimpl::trackChanges()
+{
+       Buffer * buf(bv_->buffer());
+       bool const tracking(buf->params.tracking_changes);
+
+       if (!tracking) {
+               ParIterator const end = buf->par_iterator_end();
+               for (ParIterator it = buf->par_iterator_begin(); it != end; ++it) {
+                       (*it)->trackChanges();
+               }
+               buf->params.tracking_changes = true;
+
+               // we cannot allow undos beyond the freeze point
+               buf->undostack.clear();
        } else {
-               ostringstream str;
-               str << _("Could not insert document") << ' '
-                   << disp_fn;
-               owner_->message(str.str().c_str());
+               update(BufferView::SELECT);
+               bv_->text->setCursor(buf->paragraphs.begin(), 0);
+#warning changes FIXME
+               //moveCursorUpdate(false);
+
+               bool found = lyxfind::findNextChange(bv_);
+               if (found) {
+                       owner_->getDialogs().show("changes");
+                       return;
+               }
+
+               ParIterator const end = buf->par_iterator_end();
+               for (ParIterator it = buf->par_iterator_begin(); it != end; ++it) {
+                       (*it)->untrackChanges();
+               }
+               buf->params.tracking_changes = false;
        }
+
+       buf->redostack.clear();
 }
 
 
-bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
+// Doesn't go through lyxfunc, so we need to update the
+// layout choice etc. ourselves
+bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & ev_in)
 {
+       // e.g. Qt mouse press when no buffer
+       if (!available())
+               return false;
+
+       screen().hideCursor();
+
+       bool const res = dispatch(ev_in);
+
+       // see workAreaKeyPress
+       cursor_timeout.restart();
+       screen().showCursor(*bv_);
+
+       // FIXME: we should skip these when selecting
+       bv_->owner()->updateLayoutChoice();
+       bv_->owner()->updateToolbar();
+       bv_->fitCursor();
+
+       // slight hack: this is only called currently when
+       // we clicked somewhere, so we force through the display
+       // of the new status here.
+       bv_->owner()->clearMessage();
+
+       return res;
+}
+
+
+bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
+{
+       // Make sure that the cached BufferView is correct.
+       FuncRequest ev = ev_in;
+       ev.setView(bv_);
+
        lyxerr[Debug::ACTION] << "BufferView::Pimpl::Dispatch:"
-               << " action[" << ev.action <<"]"
-               << " arg[" << ev.argument << "]"
-               << " x[" << ev.x << "]"
-               << " y[" << ev.y << "]"
-               << " button[" << ev.button() << "]"
+               << " action[" << ev.action << ']'
+               << " arg[" << ev.argument << ']'
+               << " x[" << ev.x << ']'
+               << " y[" << ev.y << ']'
+               << " button[" << ev.button() << ']'
                << endl;
 
        LyXTextClass const & tclass = buffer_->params.getLyXTextClass();
 
        switch (ev.action) {
 
-       case LFUN_TOC_INSERT:
-       {
-               InsetCommandParams p;
-               p.setCmdName("tableofcontents");
-               Inset * inset = new InsetTOC(p);
-               if (!insertInset(inset, tclass.defaultLayoutName()))
-                       delete inset;
-               break;
-       }
-
        case LFUN_SCROLL_INSET:
                // this is not handled here as this function is only active
                // if we have a locking_inset and that one is (or contains)
                // a tabular-inset
                break;
 
-       case LFUN_INSET_GRAPHICS:
-       {
-               Inset * new_inset = new InsetGraphics;
-               if (!insertInset(new_inset)) {
-                       delete new_inset;
-               } else {
-                       // this is need because you don't use a inset->Edit()
-                       updateInset(new_inset, true);
-                       new_inset->edit(bv_);
-               }
-               break;
-       }
-
-       case LFUN_LAYOUT_COPY:
-               bv_->copyEnvironment();
-               break;
-
-       case LFUN_LAYOUT_PASTE:
-               bv_->pasteEnvironment();
-               switchKeyMap();
-               break;
-
-       case LFUN_DEPTH_MIN:
-               changeDepth(bv_, bv_->getLyXText(), -1);
-               break;
-
-       case LFUN_DEPTH_PLUS:
-               changeDepth(bv_, bv_->getLyXText(), 1);
-               break;
-
-       case LFUN_FREE:
-               owner_->getDialogs().setUserFreeFont();
-               break;
-
        case LFUN_FILE_INSERT:
                MenuInsertLyXFile(ev.argument);
                break;
@@ -1026,25 +1012,15 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
                owner_->getLyXFunc().setMessage(currentState(bv_));
                break;
 
-       case LFUN_INSERT_LABEL:
-               MenuInsertLabel(bv_, ev.argument);
-               break;
-
-       case LFUN_REF_INSERT:
-               if (ev.argument.empty()) {
-                       InsetCommandParams p("ref");
-                       owner_->getDialogs().createRef(p.getAsString());
-               } else {
-                       InsetCommandParams p;
-                       p.setFromString(ev.argument);
-
-                       InsetRef * inset = new InsetRef(p, *buffer_);
-                       if (!insertInset(inset))
-                               delete inset;
-                       else
-                               updateInset(inset, true);
-               }
-               break;
+       case LFUN_INSERT_LABEL: {
+               // Try and generate a valid label
+               string const contents = ev.argument.empty() ?
+                       getPossibleLabel(*bv_) : ev.argument;
+               InsetCommandParams icp("label", contents);
+               string data = InsetCommandMailer::params2string("label", icp);
+               owner_->getDialogs().show("label", data, 0);
+       }
+       break;
 
        case LFUN_BOOKMARK_SAVE:
                savePosition(strToUnsignedInt(ev.argument));
@@ -1054,8 +1030,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
                restorePosition(strToUnsignedInt(ev.argument));
                break;
 
-       case LFUN_REF_GOTO:
-       {
+       case LFUN_REF_GOTO: {
                string label = ev.argument;
                if (label.empty()) {
                        InsetRef * inset =
@@ -1066,62 +1041,11 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
                        }
                }
 
-               if (!label.empty()) {
-                       //bv_->savePosition(0);
-                       if (!bv_->gotoLabel(label))
-                               Alert::alert(_("Error"),
-                                          _("Couldn't find this label"),
-                                          _("in current document."));
-               }
-       }
-       break;
-
-       case LFUN_HTMLURL:
-       case LFUN_URL:
-       {
-               InsetCommandParams p;
-               if (ev.action == LFUN_HTMLURL)
-                       p.setCmdName("htmlurl");
-               else
-                       p.setCmdName("url");
-               owner_->getDialogs().createUrl(p.getAsString());
+               if (!label.empty())
+                       bv_->gotoLabel(label);
        }
        break;
 
-       case LFUN_INSERT_URL:
-       {
-               InsetCommandParams p;
-               p.setFromString(ev.argument);
-
-               InsetUrl * inset = new InsetUrl(p);
-               if (!insertInset(inset))
-                       delete inset;
-               else
-                       updateInset(inset, true);
-       }
-       break;
-
-       case LFUN_INSET_CAPTION:
-       {
-               // Do we have a locking inset...
-               if (bv_->theLockingInset()) {
-                       lyxerr << "Locking inset code: "
-                              << static_cast<int>(bv_->theLockingInset()->lyxCode());
-                       InsetCaption * new_inset =
-                               new InsetCaption(buffer_->params);
-                       new_inset->setOwner(bv_->theLockingInset());
-                       new_inset->setAutoBreakRows(true);
-                       new_inset->setDrawFrame(0, InsetText::LOCKED);
-                       new_inset->setFrameColor(0, LColor::captionframe);
-                       if (insertInset(new_inset))
-                               new_inset->edit(bv_);
-                       else
-                               delete new_inset;
-               }
-       }
-       break;
-
-
        // --- accented characters ---------------------------
 
        case LFUN_UMLAUT:
@@ -1148,10 +1072,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
                        owner_->getLyXFunc().handleKeyFunc(ev.action);
                        owner_->getIntl().getTransManager()
                                .TranslateAndInsert(ev.argument[0], bv_->getLyXText());
-                       update(bv_->getLyXText(),
-                              BufferView::SELECT
-                              | BufferView::FITCUR
-                              | BufferView::CHANGE);
+                       update(bv_->getLyXText(), BufferView::SELECT);
                }
                break;
 
@@ -1162,140 +1083,90 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
        case LFUN_MATH_IMPORT_SELECTION: // Imports LaTeX from the X selection
        case LFUN_MATH_DISPLAY:          // Open or create a displayed math inset
        case LFUN_MATH_MODE:             // Open or create an inlined math inset
-       case LFUN_GREEK:                 // Insert a single greek letter
-               mathDispatch(FuncRequest(bv_, ev.action, ev.argument));
+               mathDispatch(ev);
                break;
 
-       case LFUN_CITATION_INSERT:
-       {
-               InsetCommandParams p;
-               p.setFromString(ev.argument);
-
-               InsetCitation * inset = new InsetCitation(p);
-               if (!insertInset(inset))
-                       delete inset;
-               else
-                       updateInset(inset, true);
-       }
-       break;
+       case LFUN_INSET_APPLY: {
+               string const name = ev.getArg(0);
 
-       case LFUN_INSERT_BIBTEX:
-       {
-               // ale970405+lasgoutt970425
-               // The argument can be up to two tokens separated
-               // by a space. The first one is the bibstyle.
-               string const db = token(ev.argument, ' ', 0);
-               string bibstyle = token(ev.argument, ' ', 1);
-               if (bibstyle.empty())
-                       bibstyle = "plain";
-
-               InsetCommandParams p("BibTeX", db, bibstyle);
-               InsetBibtex * inset = new InsetBibtex(p);
-
-               if (insertInset(inset)) {
-                       if (ev.argument.empty())
-                               inset->edit(bv_);
-               } else
-                       delete inset;
-       }
-       break;
-
-       // BibTeX data bases
-       case LFUN_BIBDB_ADD:
-       {
-               InsetBibtex * inset =
-                       static_cast<InsetBibtex*>(getInsetByCode(Inset::BIBTEX_CODE));
+               InsetBase * inset = owner_->getDialogs().getOpenInset(name);
                if (inset) {
-                       inset->addDatabase(ev.argument);
+                       // This works both for 'original' and 'mathed' insets.
+                       // Note that the localDispatch performs updateInset
+                       // also.
+                       FuncRequest fr(bv_, LFUN_INSET_MODIFY, ev.argument);
+                       inset->localDispatch(fr);
+               } else {
+                       FuncRequest fr(bv_, LFUN_INSET_INSERT, ev.argument);
+                       dispatch(fr);
                }
        }
        break;
 
-       case LFUN_BIBDB_DEL:
-       {
-               InsetBibtex * inset =
-                       static_cast<InsetBibtex*>(getInsetByCode(Inset::BIBTEX_CODE));
-               if (inset) {
-                       inset->delDatabase(ev.argument);
+       case LFUN_INSET_INSERT: {
+               Inset * inset = createInset(ev);
+               if (inset && insertInset(inset)) {
+                       updateInset(inset);
+
+                       string const name = ev.getArg(0);
+                       if (name == "bibitem") {
+                               // We need to do a redraw because the maximum
+                               // InsetBibitem width could have changed
+#warning please check you mean repaint() not update(),
+#warning and whether the repaint() is needed at all
+                               bv_->repaint();
+                               bv_->fitCursor();
+                       }
+               } else {
+                       delete inset;
                }
        }
        break;
 
-       case LFUN_BIBTEX_STYLE:
-       {
-               InsetBibtex * inset =
-                       static_cast<InsetBibtex*>(getInsetByCode(Inset::BIBTEX_CODE));
-               if (inset) {
-                       inset->setOptions(ev.argument);
+       case LFUN_FLOAT_LIST:
+               if (tclass.floats().typeExist(ev.argument)) {
+                       Inset * inset = new InsetFloatList(ev.argument);
+                       if (!insertInset(inset, tclass.defaultLayoutName()))
+                               delete inset;
+               } else {
+                       lyxerr << "Non-existent float type: "
+                              << ev.argument << endl;
                }
-       }
-       break;
-
-       case LFUN_INDEX_INSERT:
-       {
-               string entry = ev.argument;
-               if (entry.empty())
-                       entry = bv_->getLyXText()->getStringToIndex(bv_);
+               break;
 
-               if (entry.empty()) {
-                       owner_->getDialogs().createIndex();
+       case LFUN_LAYOUT_PARAGRAPH: {
+               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
+               if (!par)
                        break;
-               }
 
-               InsetIndex * inset = new InsetIndex(InsetCommandParams("index", entry));
+               string data;
+               params2string(*par, data);
 
-               if (!insertInset(inset)) {
-                       delete inset;
-               } else {
-                       updateInset(inset, true);
-               }
-       }
-       break;
-
-       case LFUN_INDEX_PRINT:
-       {
-               InsetCommandParams p("printindex");
-               Inset * inset = new InsetPrintIndex(p);
-               if (!insertInset(inset, tclass.defaultLayoutName()))
-                       delete inset;
+               data = "show\n" + data;
+               bv_->owner()->getDialogs().show("paragraph", data);
+               break;
        }
-       break;
 
-       case LFUN_PARENTINSERT:
-       {
-               InsetCommandParams p("lyxparent", ev.argument);
-               Inset * inset = new InsetParent(p, *buffer_);
-               if (!insertInset(inset, tclass.defaultLayoutName()))
-                       delete inset;
-       }
+       case LFUN_PARAGRAPH_UPDATE: {
+               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
+               if (!par)
+                       break;
 
-       break;
+               string data;
+               params2string(*par, data);
 
-       case LFUN_CHILD_INSERT:
-       {
-               InsetInclude::Params p;
-               p.cparams.setFromString(ev.argument);
-               p.masterFilename_ = buffer_->fileName();
+               // Will the paragraph accept changes from the dialog?
+               Inset * const inset = par->inInset();
+               bool const accept =
+                       !(inset && inset->forceDefaultParagraphs(inset));
 
-               InsetInclude * inset = new InsetInclude(p);
-               if (!insertInset(inset))
-                       delete inset;
-               else {
-                       updateInset(inset, true);
-                       bv_->owner()->getDialogs().showInclude(inset);
-               }
+               data = "update " + tostr(accept) + '\n' + data;
+               bv_->owner()->getDialogs().update("paragraph", data);
+               break;
        }
-       break;
 
-       case LFUN_FLOAT_LIST:
-               if (tclass.floats().typeExist(ev.argument)) {
-                       Inset * inset = new InsetFloatList(ev.argument);
-                       if (!insertInset(inset, tclass.defaultLayoutName()))
-                               delete inset;
-               } else {
-                       lyxerr << "Non-existent float type: "
-                              << ev.argument << endl;
-               }
+       case LFUN_PARAGRAPH_APPLY:
+               setParagraphParams(*bv_, ev.argument);
                break;
 
        case LFUN_THESAURUS_ENTRY:
@@ -1309,16 +1180,62 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
                        // FIXME
                        if (arg.size() > 100 || arg.empty()) {
                                // Get word or selection
-                               bv_->getLyXText()->selectWordWhenUnderCursor(bv_, LyXText::WHOLE_WORD);
+                               bv_->getLyXText()->selectWordWhenUnderCursor(LyXText::WHOLE_WORD);
                                arg = bv_->getLyXText()->selectionAsString(buffer_, false);
                                // FIXME: where is getLyXText()->unselect(bv_) ?
                        }
                }
 
-               bv_->owner()->getDialogs().showThesaurus(arg);
+               bv_->owner()->getDialogs().show("thesaurus", arg);
        }
                break;
 
+       case LFUN_TRACK_CHANGES:
+               trackChanges();
+               break;
+
+       case LFUN_MERGE_CHANGES:
+               owner_->getDialogs().show("changes");
+               break;
+
+       case LFUN_ACCEPT_ALL_CHANGES: {
+               update(BufferView::SELECT);
+               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
+#warning FIXME changes
+               //moveCursorUpdate(false);
+
+               while (lyxfind::findNextChange(bv_)) {
+                       bv_->getLyXText()->acceptChange();
+               }
+               update(BufferView::SELECT);
+               break;
+       }
+
+       case LFUN_REJECT_ALL_CHANGES: {
+               update(BufferView::SELECT);
+               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
+#warning FIXME changes
+               //moveCursorUpdate(false);
+
+               while (lyxfind::findNextChange(bv_)) {
+                       bv_->getLyXText()->rejectChange();
+               }
+               update(BufferView::SELECT);
+               break;
+       }
+
+       case LFUN_ACCEPT_CHANGE: {
+               bv_->getLyXText()->acceptChange();
+               update(BufferView::SELECT);
+               break;
+       }
+
+       case LFUN_REJECT_CHANGE: {
+               bv_->getLyXText()->rejectChange();
+               update(BufferView::SELECT);
+               break;
+       }
+
        case LFUN_UNKNOWN_ACTION:
                ev.errorMessage(N_("Unknown function!"));
                break;
@@ -1337,7 +1254,7 @@ bool BufferView::Pimpl::insertInset(Inset * inset, string const & lout)
        // inset there otherwise this is a illegal function now
        if (bv_->theLockingInset()) {
                if (bv_->theLockingInset()->insetAllowed(inset))
-                   return bv_->theLockingInset()->insertInset(bv_, inset);
+                       return bv_->theLockingInset()->insertInset(bv_, inset);
                return false;
        }
 
@@ -1347,15 +1264,15 @@ bool BufferView::Pimpl::insertInset(Inset * inset, string const & lout)
 
        beforeChange(bv_->text);
        if (!lout.empty()) {
-               update(bv_->text, BufferView::SELECT|BufferView::FITCUR);
-               bv_->text->breakParagraph(bv_);
-               update(bv_->text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
+               update(BufferView::SELECT);
+               bv_->text->breakParagraph(bv_->buffer()->paragraphs);
+               update(BufferView::SELECT);
 
                if (!bv_->text->cursor.par()->empty()) {
                        bv_->text->cursorLeft(bv_);
 
-                       bv_->text->breakParagraph(bv_);
-                       update(bv_->text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
+                       bv_->text->breakParagraph(bv_->buffer()->paragraphs);
+                       update(BufferView::SELECT);
                }
 
                string lres = lout;
@@ -1372,27 +1289,27 @@ bool BufferView::Pimpl::insertInset(Inset * inset, string const & lout)
                        lay = tclass.defaultLayoutName();
                }
 
-               bv_->text->setLayout(bv_, lay);
+               bv_->text->setLayout(lay);
 
-               bv_->text->setParagraph(bv_, 0, 0,
+               bv_->text->setParagraph(0, 0,
                                   0, 0,
                                   VSpace(VSpace::NONE), VSpace(VSpace::NONE),
                                   Spacing(),
                                   LYX_ALIGN_LAYOUT,
                                   string(),
                                   0);
-               update(bv_->text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
+               update(BufferView::SELECT);
        }
 
-       bv_->text->insertInset(bv_, inset);
-       update(bv_->text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
+       bv_->text->insertInset(inset);
+       update(BufferView::SELECT);
 
        unFreezeUndo();
        return true;
 }
 
 
-void BufferView::Pimpl::updateInset(Inset * inset, bool mark_dirty)
+void BufferView::Pimpl::updateInset(Inset * inset)
 {
        if (!inset || !available())
                return;
@@ -1400,20 +1317,14 @@ void BufferView::Pimpl::updateInset(Inset * inset, bool mark_dirty)
        // first check for locking insets
        if (bv_->theLockingInset()) {
                if (bv_->theLockingInset() == inset) {
-                       if (bv_->text->updateInset(bv_, inset)) {
+                       if (bv_->text->updateInset(inset)) {
                                update();
-                               if (mark_dirty) {
-                                       buffer_->markDirty();
-                               }
                                updateScrollbar();
                                return;
                        }
                } else if (bv_->theLockingInset()->updateInsetInInset(bv_, inset)) {
-                       if (bv_->text->updateInset(bv_,  bv_->theLockingInset())) {
+                       if (bv_->text->updateInset(bv_->theLockingInset())) {
                                update();
-                               if (mark_dirty) {
-                                       buffer_->markDirty();
-                               }
                                updateScrollbar();
                                return;
                        }
@@ -1426,24 +1337,16 @@ void BufferView::Pimpl::updateInset(Inset * inset, bool mark_dirty)
        Inset * tl_inset = inset;
        while (tl_inset->owner())
                tl_inset = tl_inset->owner();
-       hideCursor();
        if (tl_inset == inset) {
-               update(bv_->text, BufferView::UPDATE);
-               if (bv_->text->updateInset(bv_, inset)) {
-                       if (mark_dirty) {
-                               update(bv_->text,
-                                      BufferView::SELECT
-                                      | BufferView::FITCUR
-                                      | BufferView::CHANGE);
-                       } else {
-                               update(bv_->text, SELECT);
-                       }
+               update(BufferView::UPDATE);
+               if (bv_->text->updateInset(inset)) {
+                       update(BufferView::SELECT);
                        return;
                }
        } else if (static_cast<UpdatableInset *>(tl_inset)
                           ->updateInsetInInset(bv_, inset))
        {
-               if (bv_->text->updateInset(bv_, tl_inset)) {
+               if (bv_->text->updateInset(tl_inset)) {
                        update();
                        updateScrollbar();
                }