]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView_pimpl.C
"Inter-word Space"
[lyx.git] / src / BufferView_pimpl.C
index 51a1d488180968d439593ee17ecd39241ef9fa2c..d167010f8ed9f39a8cbb46a3863c64f421d6eff5 100644 (file)
@@ -10,6 +10,7 @@
 
 #include "BufferView_pimpl.h"
 #include "bufferlist.h"
+#include "buffer.h"
 #include "bufferview_funcs.h"
 #include "lfuns.h"
 #include "debug.h"
@@ -19,6 +20,7 @@
 #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 "graphics/Previews.h"
 
 #include "support/LAssert.h"
-#include "support/lstrings.h"
+#include "support/tostr.h"
 #include "support/filetools.h"
 
 #include <boost/bind.hpp>
 #include <boost/signals/connection.hpp>
-#include "support/BoostFormat.h"
 
 #include <unistd.h>
 #include <sys/wait.h>
@@ -259,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;
@@ -310,9 +311,13 @@ int BufferView::Pimpl::resizeCurrentBuffer()
                        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.
@@ -337,6 +342,9 @@ int BufferView::Pimpl::resizeCurrentBuffer()
        switchKeyMap();
        owner_->busy(false);
 
+       // reset the "Formatting..." message
+       owner_->clearMessage();
+
        updateScrollbar();
 
        return 0;
@@ -374,6 +382,8 @@ void BufferView::Pimpl::scrollDocView(int value)
        if (!buffer_)
                return;
 
+       screen().hideCursor();
+
        screen().draw(bv_->text, bv_, value);
 
        if (!lyxrc.cursor_follows_scrollbar)
@@ -421,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_);
+       }
 }
 
 
@@ -453,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);
        }
 }
@@ -562,11 +583,7 @@ void BufferView::Pimpl::cursorToggle()
                return;
        }
 
-       if (!bv_->theLockingInset()) {
-               screen().cursorToggle(bv_);
-       } else {
-               bv_->theLockingInset()->toggleInsetCursor(bv_);
-       }
+       screen().toggleCursor(*bv_);
 
        cursor_timeout.restart();
 }
@@ -609,15 +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;
-#if USE_BOOST_FORMAT
-               str << boost::format(_("Saved bookmark %1$d")) % i;
-#else
-               str << _("Saved bookmark ") << i;
-#endif
-               owner_->message(STRCONV(str.str()));
-       }
+       if (i > 0)
+               owner_->message(bformat(_("Saved bookmark %1$s"), tostr(i)));
 }
 
 
@@ -637,23 +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(par,
-                            min(par->size(), saved_positions[i].par_pos));
+       bv_->text->setCursor(*par,
+                            min((*par)->size(), saved_positions[i].par_pos));
 
        update(BufferView::SELECT);
-       if (i > 0) {
-               ostringstream str;
-#if USE_BOOST_FORMAT
-               str << boost::format(_("Moved to bookmark %1$d")) % i;
-#else
-               str << _("Moved to bookmark ") << i;
-#endif
-               owner_->message(STRCONV(str.str()));
-       }
+       if (i > 0)
+               owner_->message(bformat(_("Moved to bookmark %1$s"), tostr(i)));
 }
 
 
@@ -695,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())
@@ -857,33 +844,13 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
        // necessary
        filename = FileSearch(string(), filename, "lyx");
 
-       string const disp_fn(MakeDisplayPath(filename));
-
-       ostringstream s1;
-#if USE_BOOST_FORMAT
-       s1 << boost::format(_("Inserting document %1$s...")) % disp_fn;
-#else
-       s1 << _("Inserting document ") << disp_fn << _("...");
-#endif
-       owner_->message(STRCONV(s1.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;
-#if USE_BOOST_FORMAT
-               str << boost::format(_("Document %1$s inserted.")) % disp_fn;
-#else
-               str << _("Document ") << disp_fn << _(" inserted.");
-#endif
-               owner_->message(STRCONV(str.str()));
-       } else {
-               ostringstream str;
-#if USE_BOOST_FORMAT
-               str << boost::format(_("Could not insert document %1$s")) % disp_fn;
-#else
-               str << _("Could not insert document ") << disp_fn;
-#endif
-               owner_->message(STRCONV(str.str()));
-       }
+       if (res)
+               owner_->message(bformat(_("Document %1$s inserted."), disp_fn));
+       else
+               owner_->message(bformat(_("Could not insert document %1$s"), disp_fn));
 }
 
 
@@ -903,7 +870,7 @@ void BufferView::Pimpl::trackChanges()
                buf->undostack.clear();
        } else {
                update(BufferView::SELECT);
-               bv_->text->setCursor(&(*buf->paragraphs.begin()), 0);
+               bv_->text->setCursor(buf->paragraphs.begin(), 0);
 #warning changes FIXME
                //moveCursorUpdate(false);
 
@@ -932,11 +899,24 @@ bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & ev_in)
        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;
 }
 
@@ -965,15 +945,6 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                // a tabular-inset
                break;
 
-       case LFUN_LAYOUT_COPY:
-               bv_->copyEnvironment();
-               break;
-
-       case LFUN_LAYOUT_PASTE:
-               bv_->pasteEnvironment();
-               switchKeyMap();
-               break;
-
        case LFUN_FILE_INSERT:
                MenuInsertLyXFile(ev.argument);
                break;
@@ -1164,7 +1135,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                break;
 
        case LFUN_LAYOUT_PARAGRAPH: {
-               Paragraph const * par = bv_->getLyXText()->cursor.par();
+               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
                if (!par)
                        break;
 
@@ -1177,7 +1148,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
        }
 
        case LFUN_PARAGRAPH_UPDATE: {
-               Paragraph const * par = bv_->getLyXText()->cursor.par();
+               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
                if (!par)
                        break;
 
@@ -1229,7 +1200,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
 
        case LFUN_ACCEPT_ALL_CHANGES: {
                update(BufferView::SELECT);
-               bv_->text->setCursor(&(*bv_->buffer()->paragraphs.begin()), 0);
+               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
 #warning FIXME changes
                //moveCursorUpdate(false);
 
@@ -1242,7 +1213,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
 
        case LFUN_REJECT_ALL_CHANGES: {
                update(BufferView::SELECT);
-               bv_->text->setCursor(&(*bv_->buffer()->paragraphs.begin()), 0);
+               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
 #warning FIXME changes
                //moveCursorUpdate(false);
 
@@ -1366,7 +1337,6 @@ void BufferView::Pimpl::updateInset(Inset * inset)
        Inset * tl_inset = inset;
        while (tl_inset->owner())
                tl_inset = tl_inset->owner();
-       hideCursor();
        if (tl_inset == inset) {
                update(BufferView::UPDATE);
                if (bv_->text->updateInset(inset)) {