]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView.C
get rid of MSVC warning (signed/unsigned comparison)
[lyx.git] / src / BufferView.C
index ea667c43f95b50cac6bfcfa7fd202772c3fd15f6..008f03e4aa11e10347289ed7911525d6fbb5aa85 100644 (file)
@@ -20,6 +20,7 @@
 #include "buffer_funcs.h"
 #include "bufferlist.h"
 #include "bufferparams.h"
+#include "bufferview_funcs.h"
 #include "coordcache.h"
 #include "CutAndPaste.h"
 #include "debug.h"
@@ -62,6 +63,7 @@
 #include "frontends/Alert.h"
 #include "frontends/FileDialog.h"
 #include "frontends/FontMetrics.h"
+#include "frontends/Selection.h"
 
 #include "graphics/Previews.h"
 
@@ -83,10 +85,11 @@ namespace lyx {
 using support::addPath;
 using support::bformat;
 using support::FileFilterList;
+using support::FileName;
 using support::fileSearch;
 using support::isDirWriteable;
+using support::isFileReadable;
 using support::makeDisplayPath;
-using support::makeAbsPath;
 using support::package;
 
 using std::distance;
@@ -103,9 +106,6 @@ namespace Alert = frontend::Alert;
 
 namespace {
 
-unsigned int const saved_positions_num = 20;
-
-
 /// Return an inset of this class if it exists at the current cursor position
 template <class T>
 T * getInsetByCode(LCursor & cur, InsetBase::Code code)
@@ -123,23 +123,12 @@ T * getInsetByCode(LCursor & cur, InsetBase::Code code)
 
 
 BufferView::BufferView()
-       : buffer_(0), wh_(0), width_(0), height_(0),
+       : width_(0), height_(0), buffer_(0), wh_(0),
          cursor_(*this),
          multiparsel_cache_(false), anchor_ref_(0), offset_ref_(0),
-         intl_(new Intl)
+         intl_(new Intl), last_inset_(0)
 {
        xsel_cache_.set = false;
-
-       saved_positions.resize(saved_positions_num);
-       // load saved bookmarks
-       Session::BookmarkList & bmList = LyX::ref().session().loadBookmarks();
-       for (Session::BookmarkList::iterator bm = bmList.begin();
-               bm != bmList.end(); ++bm)
-               if (bm->get<0>() < saved_positions_num)
-                       saved_positions[bm->get<0>()] = Position( bm->get<1>(), bm->get<2>(), bm->get<3>() );
-       // and then clear them
-       bmList.clear();
-
        intl_->initKeyMapper(lyxrc.use_kbmap);
 }
 
@@ -167,7 +156,7 @@ void BufferView::setBuffer(Buffer * b)
                buffer_->saveCursor(cursor_.selectionBegin(),
                                    cursor_.selectionEnd());
                // current buffer is going to be switched-off, save cursor pos
-               LyX::ref().session().saveFilePosition(buffer_->fileName(),
+               LyX::ref().session().lastFilePos().save(FileName(buffer_->fileName()),
                        boost::tie(cursor_.pit(), cursor_.pos()) );
        }
 
@@ -199,7 +188,6 @@ void BufferView::setBuffer(Buffer * b)
                                    << "Buffer addr: " << buffer_ << endl;
                cursor_.push(buffer_->inset());
                cursor_.resetAnchor();
-               buffer_->text().init(this);
                buffer_->text().setCurrentFont(cursor_);
                if (buffer_->getCursor().size() > 0 &&
                    buffer_->getAnchor().size() > 0)
@@ -208,30 +196,23 @@ void BufferView::setBuffer(Buffer * b)
                        cursor_.resetAnchor();
                        cursor_.setCursor(buffer_->getCursor().asDocIterator(&(buffer_->inset())));
                        cursor_.setSelection();
+                       theSelection().haveSelection(cursor_.selection());
                }
        }
 
-       update();
+       if (buffer_)
+               updateMetrics(false);    
 
        if (buffer_ && graphics::Previews::status() != LyXRC::PREVIEW_OFF)
                graphics::Previews::get().generateBufferPreviews(*buffer_);
 }
 
 
-bool BufferView::loadLyXFile(string const & filename, bool tolastfiles)
+bool BufferView::loadLyXFile(FileName const & filename, bool tolastfiles)
 {
-       // Get absolute path of file and add ".lyx"
-       // to the filename if necessary
-       string s = fileSearch(string(), filename, "lyx");
-
-       bool const found = !s.empty();
-
-       if (!found)
-               s = filename;
-
        // File already open?
-       if (theBufferList().exists(s)) {
-               docstring const file = makeDisplayPath(s, 20);
+       if (theBufferList().exists(filename.absFilename())) {
+               docstring const file = makeDisplayPath(filename.absFilename(), 20);
                docstring text = bformat(_("The document %1$s is already "
                                                     "loaded.\n\nDo you want to revert "
                                                     "to the saved version?"), file);
@@ -239,32 +220,33 @@ bool BufferView::loadLyXFile(string const & filename, bool tolastfiles)
                        text, 0, 1,  _("&Revert"), _("&Switch to document"));
 
                if (ret != 0) {
-                       setBuffer(theBufferList().getBuffer(s));
+                       setBuffer(theBufferList().getBuffer(filename.absFilename()));
                        return true;
                }
                // FIXME: should be LFUN_REVERT
-               if (!theBufferList().close(theBufferList().getBuffer(s), false))
+               if (!theBufferList().close(theBufferList().getBuffer(filename.absFilename()), false))
                        return false;
                // Fall through to new load. (Asger)
+               buffer_ = 0;
        }
 
        Buffer * b = 0;
 
-       if (found) {
-               b = theBufferList().newBuffer(s);
-               if (!lyx::loadLyXFile(b, s)) {
+       if (isFileReadable(filename)) {
+               b = theBufferList().newBuffer(filename.absFilename());
+               if (!lyx::loadLyXFile(b, filename)) {
                        theBufferList().release(b);
                        return false;
                }
        } else {
                docstring text = bformat(_("The document %1$s does not yet "
                                                     "exist.\n\nDo you want to create "
-                                                    "a new document?"), from_utf8(s));
+                                                    "a new document?"), from_utf8(filename.absFilename()));
                int const ret = Alert::prompt(_("Create new document?"),
                         text, 0, 1, _("&Create"), _("Cancel"));
 
                if (ret == 0) {
-                       b = newFile(s, string(), true);
+                       b = newFile(filename.absFilename(), string(), true);
                        if (!b)
                                return false;
                } else
@@ -275,43 +257,27 @@ bool BufferView::loadLyXFile(string const & filename, bool tolastfiles)
        // Send the "errors" signal in case of parsing errors
        b->errors("Parse");
 
+       // Update the labels and section numbering.
+       updateLabels(*buffer_);
        // scroll to the position when the file was last closed
        if (lyxrc.use_lastfilepos) {
                pit_type pit;
                pos_type pos;
-               boost::tie(pit, pos) = LyX::ref().session().loadFilePosition(s);
-               // I am not sure how to separate the following part to a function
-               // so I will leave this to Lars.
-               //
-               // check pit since the document may be externally changed.
-               if ( static_cast<size_t>(pit) < b->paragraphs().size() ) {
-                       ParIterator it = b->par_iterator_begin();
-                       ParIterator const end = b->par_iterator_end();
-                       for (; it != end; ++it)
-                               if (it.pit() == pit) {
-                                       // restored pos may be bigger than it->size
-                                       setCursor(makeDocIterator(it, min(pos, it->size())));
-                                       update(Update::FitCursor);
-                                       break;
-                               }
+               boost::tie(pit, pos) = LyX::ref().session().lastFilePos().load(filename);
+               // if successfully move to pit (returned par_id is not zero), update metrics
+               if (moveToPosition(pit, 0, pos).get<1>()) {
+                       if (fitCursor())
+                               updateMetrics(false);
                }
        }
 
        if (tolastfiles)
-               LyX::ref().session().addLastFile(b->fileName());
+               LyX::ref().session().lastFiles().add(FileName(b->fileName()));
 
        return true;
 }
 
 
-void BufferView::reload()
-{
-       string const fn = buffer_->fileName();
-       if (theBufferList().close(buffer_, false))
-               loadLyXFile(fn);
-}
-
-
 void BufferView::resize()
 {
        if (!buffer_)
@@ -319,8 +285,7 @@ void BufferView::resize()
 
        lyxerr[Debug::DEBUG] << BOOST_CURRENT_FUNCTION << endl;
 
-       buffer_->text().init(this);
-       update();
+       updateMetrics(false);
        switchKeyMap();
 }
 
@@ -354,6 +319,10 @@ bool BufferView::multiParSel()
 
 bool BufferView::update(Update::flags flags)
 {
+       // last_inset_ points to the last visited inset. This pointer may become
+       // invalid because of keyboard editing. Since all such operations
+       // causes screen update(), I reset last_inset_ to avoid such a problem.
+       last_inset_ = 0;
        // This is close to a hot-path.
        if (lyxerr.debugging(Debug::DEBUG)) {
                lyxerr[Debug::DEBUG]
@@ -368,20 +337,58 @@ bool BufferView::update(Update::flags flags)
        if (!buffer_)
                return false;
 
-       lyxerr[Debug::WORKAREA] << "BufferView::update" << std::endl;
+       if (lyxerr.debugging(Debug::WORKAREA)) {
+               lyxerr[Debug::WORKAREA] << "BufferView::update" << std::endl;
+       }
 
        // Update macro store
        buffer_->buildMacros();
 
-       // First drawing step
-       updateMetrics(flags & Update::SinglePar);
-
+       // Now do the first drawing step if needed. This consists on updating
+       // the CoordCache in updateMetrics().
        // The second drawing step is done in WorkArea::redraw() if needed.
-       bool const need_second_step =
-               (flags & (Update::Force | Update::FitCursor | Update::MultiParSel))
-               && (fitCursor() || multiParSel());
 
-       return need_second_step;
+       // Case when no explicit update is requested.
+       if (!flags) {
+               // no need to redraw anything.
+               metrics_info_.update_strategy = NoScreenUpdate;
+               return false;
+       }
+
+       if (flags == Update::Decoration) {
+               metrics_info_.update_strategy = DecorationUpdate;
+               return true;
+       }
+
+       if (flags == Update::FitCursor 
+               || flags == (Update::Decoration | Update::FitCursor)) {
+               bool const fit_cursor = fitCursor();
+               // tell the frontend to update the screen if needed.
+               if (fit_cursor) {
+                       updateMetrics(false);
+                       return true;
+               }
+               if (flags & Update::Decoration) {
+                       metrics_info_.update_strategy = DecorationUpdate;
+                       return true;
+               }
+               // no screen update is needed.
+               metrics_info_.update_strategy = NoScreenUpdate;
+               return false;
+       }
+
+       bool full_metrics = flags & Update::Force;
+       if (flags & Update::MultiParSel)
+               full_metrics |= multiParSel();
+
+       bool const single_par = !full_metrics;
+       updateMetrics(single_par);
+
+       if (flags & Update::FitCursor && fitCursor())
+               updateMetrics(false);
+
+       // tell the frontend to update the screen.
+       return true;
 }
 
 
@@ -395,17 +402,21 @@ void BufferView::updateScrollbar()
        }
 
        LyXText & t = buffer_->text();
+       TextMetrics & tm = text_metrics_[&t];
+
        int const parsize = int(t.paragraphs().size() - 1);
        if (anchor_ref_ >  parsize)  {
                anchor_ref_ = parsize;
                offset_ref_ = 0;
        }
 
-       lyxerr[Debug::GUI]
-               << BOOST_CURRENT_FUNCTION
-               << " Updating scrollbar: height: " << t.paragraphs().size()
-               << " curr par: " << cursor_.bottom().pit()
-               << " default height " << defaultRowHeight() << endl;
+       if (lyxerr.debugging(Debug::GUI)) {
+               lyxerr[Debug::GUI]
+                       << BOOST_CURRENT_FUNCTION
+                       << " Updating scrollbar: height: " << t.paragraphs().size()
+                       << " curr par: " << cursor_.bottom().pit()
+                       << " default height " << defaultRowHeight() << endl;
+       }
 
        // It would be better to fix the scrollbar to understand
        // values in [0..1] and divide everything by wh
@@ -413,13 +424,14 @@ void BufferView::updateScrollbar()
        // estimated average paragraph height:
        if (wh_ == 0)
                wh_ = height_ / 4;
-       int h = t.getPar(anchor_ref_).height();
+
+       int h = tm.parMetrics(anchor_ref_).height();
 
        // Normalize anchor/offset (MV):
        while (offset_ref_ > h && anchor_ref_ < parsize) {
                anchor_ref_++;
                offset_ref_ -= h;
-               h = t.getPar(anchor_ref_).height();
+               h = tm.parMetrics(anchor_ref_).height();
        }
        // Look at paragraph heights on-screen
        int sumh = 0;
@@ -427,7 +439,7 @@ void BufferView::updateScrollbar()
        for (pit_type pit = anchor_ref_; pit <= parsize; ++pit) {
                if (sumh > height_)
                        break;
-               int const h2 = t.getPar(pit).height();
+               int const h2 = tm.parMetrics(pit).height();
                sumh += h2;
                nh++;
        }
@@ -460,15 +472,18 @@ void BufferView::scrollDocView(int value)
                return;
 
        LyXText & t = buffer_->text();
+       TextMetrics & tm = text_metrics_[&t];
 
        float const bar = value / float(wh_ * t.paragraphs().size());
 
        anchor_ref_ = int(bar * t.paragraphs().size());
        if (anchor_ref_ >  int(t.paragraphs().size()) - 1)
                anchor_ref_ = int(t.paragraphs().size()) - 1;
-       t.redoParagraph(anchor_ref_);
-       int const h = t.getPar(anchor_ref_).height();
+
+       tm.redoParagraph(anchor_ref_);
+       int const h = tm.parMetrics(anchor_ref_).height();
        offset_ref_ = int((bar * t.paragraphs().size() - anchor_ref_) * h);
+       updateMetrics(false);
 }
 
 
@@ -513,74 +528,49 @@ Change const BufferView::getCurrentChange() const
 }
 
 
-void BufferView::savePosition(unsigned int i)
+void BufferView::saveBookmark(bool persistent)
 {
-       if (i >= saved_positions_num)
-               return;
-       BOOST_ASSERT(cursor_.inTexted());
-       saved_positions[i] = Position(buffer_->fileName(),
-                                     cursor_.paragraph().id(),
-                                     cursor_.pos());
-       if (i > 0)
+       LyX::ref().session().bookmarks().save(
+               FileName(buffer_->fileName()),
+               cursor_.pit(),
+               cursor_.paragraph().id(),
+               cursor_.pos(),
+               persistent
+       );
+       if (persistent)
                // emit message signal.
-               message(bformat(_("Saved bookmark %1$d"), i));
+               message(_("Save bookmark"));
 }
 
 
-void BufferView::restorePosition(unsigned int i)
+boost::tuple<pit_type, int> BufferView::moveToPosition(pit_type par_pit, int par_id, pos_type par_pos)
 {
-       if (i >= saved_positions_num)
-               return;
-
-       string const fname = saved_positions[i].filename;
-
        cursor_.clearSelection();
 
-       if (fname != buffer_->fileName()) {
-               Buffer * b = 0;
-               if (theBufferList().exists(fname))
-                       b = theBufferList().getBuffer(fname);
-               else {
-                       b = theBufferList().newBuffer(fname);
-                       // Don't ask, just load it
-                       lyx::loadLyXFile(b, fname);
+       // if a valid par_id is given, try it first
+       if (par_id > 0) {
+               ParIterator par = buffer_->getParFromID(par_id);
+               if (par != buffer_->par_iterator_end()) {
+                       setCursor(makeDocIterator(par, min(par->size(), par_pos)));
+                       return boost::make_tuple(cursor_.pit(), par_id);
                }
-               if (b)
-                       setBuffer(b);
        }
-
-       ParIterator par = buffer_->getParFromID(saved_positions[i].par_id);
-       if (par == buffer_->par_iterator_end())
-               return;
-
-       setCursor(makeDocIterator(par, min(par->size(), saved_positions[i].par_pos)));
-
-       if (i > 0)
-               // emit message signal.
-               message(bformat(_("Moved to bookmark %1$d"), i));
-}
-
-
-bool BufferView::isSavedPosition(unsigned int i)
-{
-       return i < saved_positions_num && !saved_positions[i].filename.empty();
-}
-
-void BufferView::saveSavedPositions()
-{
-       // save bookmarks. It is better to use the pit interface
-       // but I do not know how to effectively convert between
-       // par_id and pit.
-       for (unsigned int i=1; i < saved_positions_num; ++i) {
-               if ( isSavedPosition(i) )
-                       LyX::ref().session().saveBookmark( boost::tie(
-                               i,
-                               saved_positions[i].filename,
-                               saved_positions[i].par_id,
-                               saved_positions[i].par_pos) );
+       // if par_id == 0, or searching through par_id failed
+       if (static_cast<size_t>(par_pit) < buffer_->paragraphs().size()) {
+               ParIterator it = buffer_->par_iterator_begin();
+               ParIterator const end = buffer_->par_iterator_end();
+               for (; it != end; ++it)
+                       if (it.pit() == par_pit) {
+                               // restored pos may be bigger than it->size
+                               setCursor(makeDocIterator(it, min(par_pos, it->size())));
+                               return boost::make_tuple(par_pit, it->id());
+                       }
        }
+       // both methods fail
+       return boost::make_tuple(pit_type(0), 0);
 }
 
+
 void BufferView::switchKeyMap()
 {
        if (!lyxrc.rtl_support)
@@ -605,12 +595,13 @@ int BufferView::workWidth() const
 void BufferView::center()
 {
        CursorSlice & bot = cursor_.bottom();
+       TextMetrics & tm = text_metrics_[bot.text()];
        pit_type const pit = bot.pit();
-       bot.text()->redoParagraph(pit);
-       Paragraph const & par = bot.text()->paragraphs()[pit];
+       tm.redoParagraph(pit);
+       ParagraphMetrics const & pm = tm.parMetrics(pit);
        anchor_ref_ = pit;
        offset_ref_ = bv_funcs::coordOffset(*this, cursor_, cursor_.boundary()).y_
-               + par.ascent() - height_ / 2;
+               + pm.ascent() - height_ / 2;
 }
 
 
@@ -627,8 +618,8 @@ FuncStatus BufferView::getStatus(FuncRequest const & cmd)
                flag.enabled(!buffer_->redostack().empty());
                break;
        case LFUN_FILE_INSERT:
-       case LFUN_FILE_INSERT_ASCII_PARA:
-       case LFUN_FILE_INSERT_ASCII:
+       case LFUN_FILE_INSERT_PLAINTEXT_PARA:
+       case LFUN_FILE_INSERT_PLAINTEXT:
        case LFUN_BOOKMARK_SAVE:
                // FIXME: Actually, these LFUNS should be moved to LyXText
                flag.enabled(cursor_.inTexted());
@@ -662,10 +653,6 @@ FuncStatus BufferView::getStatus(FuncRequest const & cmd)
                break;
        }
 
-       case LFUN_BOOKMARK_GOTO:
-               flag.enabled(isSavedPosition(convert<unsigned int>(to_utf8(cmd.argument()))));
-               break;
-
        case LFUN_CHANGES_TRACK:
                flag.enabled(true);
                flag.setOnOff(buffer_->params().trackChanges);
@@ -683,7 +670,11 @@ FuncStatus BufferView::getStatus(FuncRequest const & cmd)
        case LFUN_CHANGE_NEXT:
        case LFUN_ALL_CHANGES_ACCEPT:
        case LFUN_ALL_CHANGES_REJECT:
-               flag.enabled(buffer_); // FIXME: Change tracking (MG)
+               // TODO: context-sensitive enabling of LFUNs
+               // In principle, these command should only be enabled if there
+               // is a change in the document. However, without proper
+               // optimizations, this will inevitably result in poor performance.
+               flag.enabled(buffer_);
                break;
 
        case LFUN_BUFFER_TOGGLE_COMPRESSION: {
@@ -740,30 +731,26 @@ bool BufferView::dispatch(FuncRequest const & cmd)
                break;
 
        case LFUN_FILE_INSERT:
-               // FIXME: We don't know the encoding of filenames
+               // FIXME UNICODE
                menuInsertLyXFile(to_utf8(cmd.argument()));
                break;
 
-       case LFUN_FILE_INSERT_ASCII_PARA:
-               // FIXME: We don't know the encoding of filenames
-               insertAsciiFile(this, to_utf8(cmd.argument()), true);
+       case LFUN_FILE_INSERT_PLAINTEXT_PARA:
+               // FIXME UNICODE
+               insertPlaintextFile(this, to_utf8(cmd.argument()), true);
                break;
 
-       case LFUN_FILE_INSERT_ASCII:
-               // FIXME: We don't know the encoding of filenames
-               insertAsciiFile(this, to_utf8(cmd.argument()), false);
+       case LFUN_FILE_INSERT_PLAINTEXT:
+               // FIXME UNICODE
+               insertPlaintextFile(this, to_utf8(cmd.argument()), false);
                break;
 
        case LFUN_FONT_STATE:
-               cur.message(from_utf8(cur.currentState()));
+               cur.message(cur.currentState());
                break;
 
        case LFUN_BOOKMARK_SAVE:
-               savePosition(convert<unsigned int>(to_utf8(cmd.argument())));
-               break;
-
-       case LFUN_BOOKMARK_GOTO:
-               restorePosition(convert<unsigned int>(to_utf8(cmd.argument())));
+               saveBookmark(convert<unsigned int>(to_utf8(cmd.argument())));
                break;
 
        case LFUN_LABEL_GOTO: {
@@ -774,7 +761,8 @@ bool BufferView::dispatch(FuncRequest const & cmd)
                                                         InsetBase::REF_CODE);
                        if (inset) {
                                label = inset->getParam("reference");
-                               savePosition(0);
+                               // persistent=false: use temp_bookmark
+                               saveBookmark(false);
                        }
                }
 
@@ -785,21 +773,34 @@ bool BufferView::dispatch(FuncRequest const & cmd)
 
        case LFUN_PARAGRAPH_GOTO: {
                int const id = convert<int>(to_utf8(cmd.argument()));
-               ParIterator par = buffer_->getParFromID(id);
-               if (par == buffer_->par_iterator_end()) {
-                       lyxerr[Debug::INFO] << "No matching paragraph found! ["
-                                           << id << ']' << endl;
-                       break;
-               } else {
-                       lyxerr[Debug::INFO] << "Paragraph " << par->id()
-                                           << " found." << endl;
+               int i = 0;
+               for (Buffer * b = buffer_; i == 0 || b != buffer_; b = theBufferList().next(b)) {
+                       ParIterator par = b->getParFromID(id);
+                       if (par == b->par_iterator_end()) {
+                               lyxerr[Debug::INFO]
+                                       << "No matching paragraph found! ["
+                                       << id << "]." << endl;
+                       } else {
+                               lyxerr[Debug::INFO]
+                                       << "Paragraph " << par->id()
+                                       << " found in buffer `"
+                                       << b->fileName() << "'." << endl;
+
+                               if (b == buffer_) {
+                                       // Set the cursor
+                                       setCursor(makeDocIterator(par, 0));
+                                       update();
+                                       switchKeyMap();
+                               } else {
+                                       // Switch to other buffer view and resend cmd
+                                       theLyXFunc().dispatch(FuncRequest(
+                                               LFUN_BUFFER_SWITCH, b->fileName()));
+                                       theLyXFunc().dispatch(cmd);
+                               }
+                               break;
+                       }
+                       ++i;
                }
-
-               // Set the cursor
-               setCursor(makeDocIterator(par, 0));
-
-               update();
-               switchKeyMap();
                break;
        }
 
@@ -858,7 +859,7 @@ bool BufferView::dispatch(FuncRequest const & cmd)
 #warning FIXME changes
 #endif
                while (findNextChange(this))
-                       getLyXText()->acceptChange(cursor_);
+                       getLyXText()->acceptOrRejectChange(cursor_, true);
                update();
                break;
        }
@@ -869,7 +870,7 @@ bool BufferView::dispatch(FuncRequest const & cmd)
 #warning FIXME changes
 #endif
                while (findNextChange(this))
-                       getLyXText()->rejectChange(cursor_);
+                       getLyXText()->acceptOrRejectChange(cursor_, false);
                break;
        }
 
@@ -1031,26 +1032,24 @@ void BufferView::clearSelection()
        if (buffer_) {
                cursor_.clearSelection();
                xsel_cache_.set = false;
+               // The buffer did not really change, but this causes the
+               // redraw we need because we cleared the selection above.
+               buffer_->changed();
        }
 }
 
 
 void BufferView::workAreaResize(int width, int height)
 {
-       bool const widthChange = width != width_;
-       bool const heightChange = height != height_;
-
        // Update from work area
        width_ = width;
        height_ = height;
 
-       if (buffer_ && widthChange) {
-               // The WorkArea content needs a resize
-               resize();
-       }
+       // The complete text metrics will be redone.
+       text_metrics_.clear();
 
-       if (widthChange || heightChange)
-               update();
+       if (buffer_)
+               resize();
 }
 
 
@@ -1062,6 +1061,7 @@ bool BufferView::workAreaDispatch(FuncRequest const & cmd0)
        // LFUN_FILE_OPEN generated by drag-and-drop.
        FuncRequest cmd = cmd0;
 
+       // E.g. Qt mouse press when no buffer
        if (!buffer_)
                return false;
 
@@ -1069,24 +1069,65 @@ bool BufferView::workAreaDispatch(FuncRequest const & cmd0)
        cur.push(buffer_->inset());
        cur.selection() = cursor_.selection();
 
-       // Doesn't go through lyxfunc, so we need to update
-       // the layout choice etc. ourselves
-
-       // E.g. Qt mouse press when no buffer
-       if (!buffer_)
-               return false;
-
        // Either the inset under the cursor or the
        // surrounding LyXText will handle this event.
 
        // Build temporary cursor.
        cmd.y = min(max(cmd.y, -1), height_);
        InsetBase * inset = buffer_->text().editXY(cur, cmd.x, cmd.y);
+
        //lyxerr << BOOST_CURRENT_FUNCTION
        //       << " * hit inset at tip: " << inset << endl;
        //lyxerr << BOOST_CURRENT_FUNCTION
        //       << " * created temp cursor:" << cur << endl;
 
+       // NOTE: editXY returns the top level inset of nested insets. If you happen
+       // to move from a text (inset=0) to a text inside an inset (e.g. an opened
+       // footnote inset, again inset=0), that inset will not be redrawn.
+       if (cmd.action == LFUN_MOUSE_MOTION && cmd.button() == mouse_button::none) {
+               bool need_redraw = false;
+               
+               if (inset != last_inset_) {
+                       if (last_inset_)
+                               need_redraw |= last_inset_->setMouseHover(false);
+                       if (inset)
+                               need_redraw |= inset->setMouseHover(true);
+                       last_inset_ = inset;
+               }
+
+               // if last metrics update was in singlepar mode, WorkArea::redraw() will
+               // not expose the button for redraw. We adjust here the metrics dimension
+               // to enable a full redraw.
+               // FIXME: It is possible to redraw only the area around the button!
+               if (need_redraw 
+                       && metrics_info_.update_strategy == SingleParUpdate) {
+                       // FIXME: It should be possible to redraw only the area around 
+                       // the button by doing this:
+                       //
+                       //metrics_info_.singlepar = false;
+                       //metrics_info_.y1 = ymin of button;
+                       //metrics_info_.y2 = ymax of button;
+                       //
+                       // Unfortunately, rowpainter.C:paintText() does not distinguish
+                       // between background updates and text updates. So we use the hammer
+                       // solution for now. We could also avoid the updateMetrics() below
+                       // by using the first and last pit of the CoordCache. Have a look
+                       // at LyXText::getPitNearY() to see what I mean.
+                       //
+                       //metrics_info_.pit1 = first pit of CoordCache;
+                       //metrics_info_.pit2 = last pit of CoordCache;
+                       //metrics_info_.singlepar = false;
+                       //metrics_info_.y1 = 0;
+                       //metrics_info_.y2 = height_;
+                       //
+                       updateMetrics(false);
+               }
+
+               // This event (moving without mouse click) is not passed further.
+               // This should be changed if it is further utilized.
+               return need_redraw;
+       }
+
        // Put anchor at the same position.
        cur.resetAnchor();
 
@@ -1094,23 +1135,20 @@ bool BufferView::workAreaDispatch(FuncRequest const & cmd0)
        // via the temp cursor. If the inset wishes to change the real
        // cursor it has to do so explicitly by using
        //  cur.bv().cursor() = cur;  (or similar)
-       if (inset)
+       if (inset) {
                inset->dispatch(cur, cmd);
+       }
 
        // Now dispatch to the temporary cursor. If the real cursor should
        // be modified, the inset's dispatch has to do so explicitly.
        if (!cur.result().dispatched())
                cur.dispatch(cmd);
 
-       if (cur.result().dispatched()) {
-               // Redraw if requested or necessary.
-               if (cur.result().update())
-                       update(Update::FitCursor | Update::Force);
-               else
-                       update(Update::FitCursor | Update::MultiParSel);
-       }
+       // Redraw if requested and necessary.
+       if (cur.result().dispatched() && cur.result().update())
+               return update(cur.result().update());
 
-       return true;
+       return false;
 }
 
 
@@ -1131,8 +1169,6 @@ void BufferView::scroll(int /*lines*/)
 //
 //     scrollDocView(new_top_y);
 //
-//     // Update the scrollbar.
-//     workArea_->setScrollbarParams(t->height(), top_y(), defaultRowHeight());}
 }
 
 
@@ -1180,6 +1216,30 @@ LyXText const * BufferView::getLyXText() const
 }
 
 
+TextMetrics const & BufferView::textMetrics(LyXText const * t) const
+{
+       return const_cast<BufferView *>(this)->textMetrics(t);
+}
+
+
+TextMetrics & BufferView::textMetrics(LyXText const * t)
+{
+       TextMetricsCache::iterator tmc_it  = text_metrics_.find(t);
+       if (tmc_it == text_metrics_.end()) {
+               tmc_it = text_metrics_.insert(
+                       make_pair(t, TextMetrics(this, const_cast<LyXText *>(t)))).first;
+       }       
+       return tmc_it->second;
+}
+
+
+ParagraphMetrics const & BufferView::parMetrics(LyXText const * t,
+               pit_type pit) const
+{
+       return textMetrics(t).parMetrics(pit);
+}
+
+
 int BufferView::workHeight() const
 {
        return height_;
@@ -1197,7 +1257,29 @@ void BufferView::setCursor(DocIterator const & dit)
 }
 
 
-void BufferView::mouseSetCursor(LCursor & cur)
+bool BufferView::checkDepm(LCursor & cur, LCursor & old)
+{
+       // Would be wrong to delete anything if we have a selection.
+       if (cur.selection())
+               return false;
+
+       bool need_anchor_change = false;
+       bool changed = cursor_.text()->deleteEmptyParagraphMechanism(cur, old,
+               need_anchor_change);
+
+       if (need_anchor_change)
+               cur.resetAnchor();
+       
+       if (!changed)
+               return false;
+
+       updateMetrics(false);
+       buffer_->changed();
+       return true;
+}
+
+
+bool BufferView::mouseSetCursor(LCursor & cur)
 {
        BOOST_ASSERT(&cur.bv() == this);
 
@@ -1208,14 +1290,33 @@ void BufferView::mouseSetCursor(LCursor & cur)
 
        // do the dEPM magic if needed
        // FIXME: move this to InsetText::notifyCursorLeaves?
+       bool update = false;
        if (!badcursor && cursor_.inTexted())
-               cursor_.text()->deleteEmptyParagraphMechanism(cur, cursor_);
+               checkDepm(cur, cursor_);
+
+       // if the cursor was in an empty script inset and the new
+       // position is in the nucleus of the inset, notifyCursorLeaves
+       // will kill the script inset itself. So we check all the
+       // elements of the cursor to make sure that they are correct.
+       // For an example, see bug 2933: 
+       // http://bugzilla.lyx.org/show_bug.cgi?id=2933
+       // The code below could maybe be moved to a DocIterator method.
+       //lyxerr << "cur before " << cur <<std::endl;
+       DocIterator dit(cur.inset());
+       dit.push_back(cur.bottom());
+       size_t i = 1;
+       while (i < cur.depth() && dit.nextInset() == &cur[i].inset()) {
+               dit.push_back(cur[i]);
+               ++i;
+       }
+       //lyxerr << "5 cur after" << dit <<std::endl;
 
-       cursor_ = cur;
+       cursor_.setCursor(dit);
        cursor_.clearSelection();
+       // remember new position.
        cursor_.setTargetX();
        finishUndo();
-
+       return update;
 }
 
 
@@ -1232,6 +1333,7 @@ void BufferView::putSelectionAt(DocIterator const & cur,
                        cursor_.setSelection(cursor_, -length);
                } else
                        cursor_.setSelection(cursor_, length);
+               theSelection().haveSelection(cursor_.selection());
        }
 }
 
@@ -1260,52 +1362,56 @@ ViewMetricsInfo const & BufferView::viewMetricsInfo()
 }
 
 
+// FIXME: We should split-up updateMetrics() for the singlepar case.
 void BufferView::updateMetrics(bool singlepar)
 {
-       // FIXME (Abdel 19/10/2006):
-       // There's something fishy in tabular. The coord_cache_ is not
-       // correctly reconstructed when a character is trying to be inserted.
-       // Not clearing out the coord_cache_ fixes the crash but there is a
-       // bad side effect: buffer-begin and buffer-end do not update the screen.
-       //
-       // Remove old position cache
-       coord_cache_.clear();
-
        LyXText & buftext = buffer_->text();
+       TextMetrics & tm = textMetrics(&buftext);
        pit_type size = int(buftext.paragraphs().size());
 
        if (anchor_ref_ > int(buftext.paragraphs().size() - 1)) {
                anchor_ref_ = int(buftext.paragraphs().size() - 1);
                offset_ref_ = 0;
        }
+       
+       // If the paragraph metrics has changed, we can not
+       // use the singlepar optimisation.
+       if (singlepar
+               // In Single Paragraph mode, rebreak only
+               // the (main text, not inset!) paragraph containing the cursor.
+               // (if this paragraph contains insets etc., rebreaking will
+               // recursively descend)
+               && tm.redoParagraph(cursor_.bottom().pit()))
+               singlepar = false;
 
        pit_type const pit = anchor_ref_;
        int pit1 = pit;
        int pit2 = pit;
        size_t const npit = buftext.paragraphs().size();
 
-       // Rebreak anchor paragraph. In Single Paragraph mode, rebreak only
-       // the (main text, not inset!) paragraph containing the cursor.
-       // (if this paragraph contains insets etc., rebreaking will
-       // recursively descend)
-       if (!singlepar || pit == cursor_.bottom().pit())
-               buftext.redoParagraph(pit);
-       int y0 = buftext.getPar(pit).ascent() - offset_ref_;
+       // Rebreak anchor paragraph.
+       if (!singlepar)
+               tm.redoParagraph(pit);
+       
+       // Clear out the position cache in case of full screen redraw.
+       if (!singlepar)
+               coord_cache_.clear();
+
+       int y0 = tm.parMetrics(pit).ascent() - offset_ref_;
 
-       // Redo paragraphs above anchor if necessary; again, in Single Par
-       // mode, only if we encounter the (main text) one having the cursor.
+       // Redo paragraphs above anchor if necessary.
        int y1 = y0;
        while (y1 > 0 && pit1 > 0) {
-               y1 -= buftext.getPar(pit1).ascent();
+               y1 -= tm.parMetrics(pit1).ascent();
                --pit1;
-               if (!singlepar || pit1 == cursor_.bottom().pit())
-                       buftext.redoParagraph(pit1);
-               y1 -= buftext.getPar(pit1).descent();
+               if (!singlepar)
+                       tm.redoParagraph(pit1);
+               y1 -= tm.parMetrics(pit1).descent();
        }
 
 
        // Take care of ascent of first line
-       y1 -= buftext.getPar(pit1).ascent();
+       y1 -= tm.parMetrics(pit1).ascent();
 
        // Normalize anchor for next time
        anchor_ref_ = pit1;
@@ -1318,34 +1424,33 @@ void BufferView::updateMetrics(bool singlepar)
                anchor_ref_ = 0;
        }
 
-       // Redo paragraphs below the anchor if necessary. Single par mode:
-       // only the one containing the cursor if encountered.
+       // Redo paragraphs below the anchor if necessary.
        int y2 = y0;
        while (y2 < height_ && pit2 < int(npit) - 1) {
-               y2 += buftext.getPar(pit2).descent();
+               y2 += tm.parMetrics(pit2).descent();
                ++pit2;
-               if (!singlepar || pit2 == cursor_.bottom().pit())
-                       buftext.redoParagraph(pit2);
-               y2 += buftext.getPar(pit2).ascent();
+               if (!singlepar)
+                       tm.redoParagraph(pit2);
+               y2 += tm.parMetrics(pit2).ascent();
        }
 
        // Take care of descent of last line
-       y2 += buftext.getPar(pit2).descent();
+       y2 += tm.parMetrics(pit2).descent();
 
        // The coordinates of all these paragraphs are correct, cache them
        int y = y1;
        CoordCache::InnerParPosCache & parPos = coord_cache_.parPos()[&buftext];
        for (pit_type pit = pit1; pit <= pit2; ++pit) {
-               Paragraph const & par = buftext.getPar(pit);
-               y += par.ascent();
+               ParagraphMetrics const & pm = tm.parMetrics(pit);
+               y += pm.ascent();
                parPos[pit] = Point(0, y);
                if (singlepar && pit == cursor_.bottom().pit()) {
                        // In Single Paragraph mode, collect here the
                        // y1 and y2 of the (one) paragraph the cursor is in
-                       y1 = y - par.ascent();
-                       y2 = y + par.descent();
+                       y1 = y - pm.ascent();
+                       y2 = y + pm.descent();
                }
-               y += par.descent();
+               y += pm.descent();
        }
 
        if (singlepar) {
@@ -1365,7 +1470,8 @@ void BufferView::updateMetrics(bool singlepar)
                << "size: " << size
                << endl;
 
-       metrics_info_ = ViewMetricsInfo(pit1, pit2, y1, y2, singlepar, size);
+       metrics_info_ = ViewMetricsInfo(pit1, pit2, y1, y2, 
+               singlepar? SingleParUpdate: FullScreenUpdate, size);
 
        if (lyxerr.debugging(Debug::WORKAREA)) {
                lyxerr[Debug::WORKAREA] << "BufferView::updateMetrics" << endl;
@@ -1387,7 +1493,7 @@ void BufferView::menuInsertLyXFile(string const & filenm)
                if (buffer_) {
                        string const trypath = buffer_->filePath();
                        // If directory is writeable, use this as default.
-                       if (isDirWriteable(trypath))
+                       if (isDirWriteable(FileName(trypath)))
                                initpath = trypath;
                }
 
@@ -1418,7 +1524,7 @@ void BufferView::menuInsertLyXFile(string const & filenm)
 
        // Get absolute path of file and add ".lyx"
        // to the filename if necessary
-       filename = fileSearch(string(), filename, "lyx");
+       filename = fileSearch(string(), filename, "lyx").absFilename();
 
        docstring const disp_fn = makeDisplayPath(filename);
        // emit message signal.
@@ -1426,10 +1532,11 @@ void BufferView::menuInsertLyXFile(string const & filenm)
 
        docstring res;
        Buffer buf("", false);
-       if (lyx::loadLyXFile(&buf, makeAbsPath(filename))) {
+       if (lyx::loadLyXFile(&buf, FileName(filename))) {
                ErrorList & el = buffer_->errorList("Parse");
                // Copy the inserted document error list into the current buffer one.
                el = buf.errorList("Parse");
+               recordUndo(cursor_);
                cap::pasteParagraphList(cursor_, buf.paragraphs(),
                                             buf.params().textclass, el);
                res = _("Document %1$s inserted.");
@@ -1442,5 +1549,4 @@ void BufferView::menuInsertLyXFile(string const & filenm)
        resize();
 }
 
-
 } // namespace lyx