]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetcollapsable.C
fix #832
[lyx.git] / src / insets / insetcollapsable.C
index 8a0ab06fb517cb1ba72e968b7fda00b0165d45c9..ea684568d03b06aedd2680d111fe71c68e19649f 100644 (file)
@@ -1,47 +1,51 @@
-/* This file is part of
- * ======================================================
- * 
- *           LyX, The Document Processor
- *      
- *          Copyright 1998-2001 The LyX Team.
+/**
+ * \file insetcollapsable.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- * ======================================================
+ * \author Alejandro Aguilar Sierra
+ * \author Jürgen Vigna
+ * \author Lars Gullik Bjønnes
+ *
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "insetcollapsable.h"
-#include "gettext.h"
-#include "lyxfont.h"
+#include "insettext.h"
+
 #include "BufferView.h"
-#include "Painter.h"
-#include "insets/insettext.h"
-#include "support/LOstream.h"
-#include "support/lstrings.h"
 #include "debug.h"
-#include "lyxtext.h"
-#include "font.h"
+#include "gettext.h"
+#include "lyxfont.h"
 #include "lyxlex.h"
+#include "lyxtext.h"
+#include "WordLangTuple.h"
+#include "funcrequest.h"
+#include "buffer.h"
 
-class LyXText;
+#include "frontends/font_metrics.h"
+#include "frontends/Painter.h"
+#include "frontends/LyXView.h"
 
+#include "support/LAssert.h"
+#include "support/LOstream.h"
+
+using std::vector;
 using std::ostream;
 using std::endl;
 using std::max;
 
 
-InsetCollapsable::InsetCollapsable(bool collapsed)
-       : UpdatableInset(), collapsed_(collapsed), 
+InsetCollapsable::InsetCollapsable(BufferParams const & bp, bool collapsed)
+       : UpdatableInset(), collapsed_(collapsed), inset(bp),
          button_length(0), button_top_y(0), button_bottom_y(0),
-         need_update(NONE), label("Label"),
+         label("Label"),
 #if 0
        autocollapse(false),
 #endif
-         oldWidth(0), in_update(false)
+         oldWidth(0), in_update(false), first_after_edit(false)
 {
        inset.setOwner(this);
        inset.setAutoBreakRows(true);
@@ -52,14 +56,14 @@ InsetCollapsable::InsetCollapsable(bool collapsed)
 
 
 InsetCollapsable::InsetCollapsable(InsetCollapsable const & in, bool same_id)
-       : UpdatableInset(in, same_id), collapsed_(in.collapsed_), 
-         framecolor(in.framecolor), labelfont(in.labelfont),
+       : UpdatableInset(in, same_id), collapsed_(in.collapsed_),
+         framecolor(in.framecolor), labelfont(in.labelfont), inset(in.inset),
          button_length(0), button_top_y(0), button_bottom_y(0),
-         need_update(NONE), label(in.label),
+         label(in.label),
 #if 0
          autocollapse(in.autocollapse),
 #endif
-         oldWidth(0), in_update(false)
+         oldWidth(0), in_update(false), first_after_edit(false)
 {
        inset.init(&(in.inset), same_id);
        inset.setOwner(this);
@@ -79,7 +83,7 @@ bool InsetCollapsable::insertInset(BufferView * bv, Inset * in)
 
 void InsetCollapsable::write(Buffer const * buf, ostream & os) const
 {
-       os << "collapsed " << tostr(collapsed_) << "\n";
+       os << "collapsed " << (collapsed_ ? "true" : "false") << "\n";
        inset.writeParagraphData(buf, os);
 }
 
@@ -109,7 +113,7 @@ int InsetCollapsable::ascent_collapsed() const
        int width = 0;
        int ascent = 0;
        int descent = 0;
-       lyxfont::buttonText(label, labelfont, width, ascent, descent);
+       font_metrics::buttonText(label, labelfont, width, ascent, descent);
        return ascent;
 }
 
@@ -119,7 +123,7 @@ int InsetCollapsable::descent_collapsed() const
        int width = 0;
        int ascent = 0;
        int descent = 0;
-       lyxfont::buttonText(label, labelfont, width, ascent, descent);
+       font_metrics::buttonText(label, labelfont, width, ascent, descent);
        return descent;
 }
 
@@ -130,8 +134,8 @@ int InsetCollapsable::width_collapsed() const
        int width;
        int ascent;
        int descent;
-       lyxfont::buttonText(label, labelfont, width, ascent, descent);
-       return width + (2*TEXT_TO_INSET_OFFSET);
+       font_metrics::buttonText(label, labelfont, width, ascent, descent);
+       return width + 2 * TEXT_TO_INSET_OFFSET;
 }
 
 
@@ -143,7 +147,7 @@ int InsetCollapsable::ascent(BufferView * /*bv*/, LyXFont const &) const
 
 int InsetCollapsable::descent(BufferView * bv, LyXFont const & font) const
 {
-       if (collapsed_) 
+       if (collapsed_)
                return descent_collapsed();
 
        return descent_collapsed()
@@ -155,13 +159,13 @@ int InsetCollapsable::descent(BufferView * bv, LyXFont const & font) const
 
 int InsetCollapsable::width(BufferView * bv, LyXFont const & font) const
 {
-       if (collapsed_) 
+       if (collapsed_)
                return width_collapsed();
 
-       int widthCollapsed = width_collapsed();
+       int const collapsed_width = width_collapsed();
+       int const contents_width = inset.width(bv, font);
 
-       return (inset.width(bv, font) > widthCollapsed) ?
-               inset.width(bv, font) : widthCollapsed;
+       return max(collapsed_width, contents_width);
 }
 
 
@@ -174,15 +178,12 @@ void InsetCollapsable::draw_collapsed(Painter & pain,
 }
 
 
-void InsetCollapsable::draw(BufferView * bv, LyXFont const & f, 
-                            int baseline, float & x, bool cleared) const
+void InsetCollapsable::draw(BufferView * bv, LyXFont const & f,
+                            int baseline, float & x, bool inlined) const
 {
-       if (need_update != NONE) {
-               const_cast<InsetText *>(&inset)->update(bv, f, true);
-               bv->text->status(bv, LyXText::CHANGED_IN_DRAW);
-               need_update = NONE;
-               return;
-       }
+       lyx::Assert(bv);
+       cache(bv);
+
        if (nodraw())
                return;
 
@@ -201,91 +202,36 @@ void InsetCollapsable::draw(BufferView * bv, LyXFont const & f,
        float old_x = x;
 
        if (!owner())
-               x += static_cast<float>(scroll());
-
-       if (!cleared && (inset.need_update == InsetText::FULL ||
-                        inset.need_update == InsetText::INIT ||
-                        top_x != int(x) ||
-                        top_baseline != baseline))
-       {
-               // we don't need anymore to clear here we just have to tell
-               // the underlying LyXText that it should do the RowClear!
-               inset.setUpdateStatus(bv, InsetText::FULL);
-               bv->text->status(bv, LyXText::CHANGED_IN_DRAW);
-               return;
-       }
+               x += scroll();
 
        top_x = int(x);
-       topx_set = true;
        top_baseline = baseline;
 
        int const bl = baseline - ascent(bv, f) + ascent_collapsed();
-       
-       draw_collapsed(pain, bl, old_x);
-       inset.draw(bv, f, 
-                          bl + descent_collapsed() + inset.ascent(bv, f),
-                          x, cleared);
-       if (x < (top_x + button_length + TEXT_TO_INSET_OFFSET))
-               x = top_x + button_length + TEXT_TO_INSET_OFFSET;
-}
-
-
-void InsetCollapsable::edit(BufferView * bv, int xp, int yp,
-                            unsigned int button)
-{
-       UpdatableInset::edit(bv, xp, yp, button);
 
-       if (collapsed_) {
-               collapsed_ = false;
-               // set this only here as it should be recollapsed only if
-               // it was already collapsed!
-               first_after_edit = true;
-               if (!bv->lockInset(this))
-                       return;
-               bv->updateInset(this, false);
-               inset.edit(bv);
+       if (inlined) {
+               inset.draw(bv, f, baseline, x);
        } else {
-               if (!bv->lockInset(this))
-                       return;
-               if (yp <= button_bottom_y) {
-                       inset.edit(bv);
-               } else {
-                       LyXFont font(LyXFont::ALL_SANE);
-                       int yy = ascent(bv, font) + yp -
-                               (ascent_collapsed() +
-                                descent_collapsed() +
-                                inset.ascent(bv, font));
-                       inset.edit(bv, xp, yy, button);
-               }
+               draw_collapsed(pain, bl, old_x);
+               inset.draw(bv, f, bl + descent_collapsed() + inset.ascent(bv, f), x);
+               // contained inset may be shorter than the button
+               if (x < (top_x + button_length + TEXT_TO_INSET_OFFSET))
+                       x = top_x + button_length + TEXT_TO_INSET_OFFSET;
        }
 }
 
 
-void InsetCollapsable::edit(BufferView * bv, bool front)
+void InsetCollapsable::draw(BufferView * bv, LyXFont const & f,
+                           int baseline, float & x) const
 {
-       UpdatableInset::edit(bv, front);
-
-       if (collapsed_) {
-               collapsed_ = false;
-               if (!bv->lockInset(this))
-                       return;
-               inset.setUpdateStatus(bv, InsetText::FULL);
-               bv->updateInset(this, false);
-               inset.edit(bv, front);
-       } else {
-               if (!bv->lockInset(this))
-                       return;
-               inset.edit(bv, front);
-       }
-       first_after_edit = true;
+       // by default, we are not inlined-drawing
+       draw(bv, f, baseline, x, false);
 }
 
 
 Inset::EDITABLE InsetCollapsable::editable() const
 {
-       if (collapsed_)
-               return IS_EDITABLE;
-       return HIGHLY_EDITABLE;
+       return collapsed_ ? IS_EDITABLE : HIGHLY_EDITABLE;
 }
 
 
@@ -304,131 +250,180 @@ void InsetCollapsable::insetUnlock(BufferView * bv)
        inset.insetUnlock(bv);
        if (scroll())
                scroll(bv, 0.0F);
-       bv->updateInset(this, false);
+       bv->updateInset(this);
 }
 
 
-void InsetCollapsable::insetButtonPress(BufferView * bv,
-                                        int x, int y, int button)
+FuncRequest InsetCollapsable::adjustCommand(FuncRequest const & cmd)
 {
-       if (!collapsed_ && (y > button_bottom_y)) {
-               LyXFont font(LyXFont::ALL_SANE);
-               int yy = ascent(bv, font) + y -
-                   (ascent_collapsed() +
-                    descent_collapsed() +
-                    inset.ascent(bv, font));
-               inset.insetButtonPress(bv, x, yy, button);
-       }
+       LyXFont font(LyXFont::ALL_SANE);
+       FuncRequest cmd1 = cmd;
+       cmd1.y = ascent(cmd.view(), font) + cmd.y -
+           (ascent_collapsed() +
+            descent_collapsed() +
+            inset.ascent(cmd.view(), font));
+       return cmd1;
 }
 
 
-bool InsetCollapsable::insetButtonRelease(BufferView * bv,
-                                          int x, int y, int button)
+void InsetCollapsable::lfunMouseRelease(FuncRequest const & cmd)
 {
        bool ret = false;
-       if ((button != 3) && (x >= 0)  && (x < button_length) &&
-           (y >= button_top_y) &&  (y <= button_bottom_y))
+       BufferView * bv = cmd.view();
+
+       if (collapsed_ && cmd.button() != mouse_button::button3) {
+               collapsed_ = false;
+               inset.setUpdateStatus(bv, InsetText::FULL);
+               bv->updateInset(this);
+               bv->buffer()->markDirty();
+               return;
+       }
+
+       if ((cmd.button() != mouse_button::button3) && (cmd.x < button_length) &&
+           (cmd.y >= button_top_y) && (cmd.y <= button_bottom_y))
        {
                if (collapsed_) {
                        collapsed_ = false;
-// should not be called on inset open!
-//                     inset.insetButtonRelease(bv, 0, 0, button);
                        inset.setUpdateStatus(bv, InsetText::FULL);
-                       bv->updateInset(this, false);
+                       bv->updateInset(this);
+                       bv->buffer()->markDirty();
                } else {
                        collapsed_ = true;
                        bv->unlockInset(this);
-                       bv->updateInset(this, false);
+                       bv->updateInset(this);
+                       bv->buffer()->markDirty();
                }
-       } else if (!collapsed_ && (y > button_bottom_y)) {
-               LyXFont font(LyXFont::ALL_SANE);
-               int yy = ascent(bv, font) + y -
-                   (ascent_collapsed() +
-                    descent_collapsed() +
-                    inset.ascent(bv, font));
-               ret = inset.insetButtonRelease(bv, x, yy, button);
-       }
-       if ((button == 3) && !ret) {
-               return showInsetDialog(bv);
+       } else if (!collapsed_ && (cmd.y > button_bottom_y)) {
+               ret = (inset.localDispatch(adjustCommand(cmd)) == DISPATCHED);
        }
-       return false;
+       if (cmd.button() == mouse_button::button3 && !ret)
+               showInsetDialog(bv);
 }
 
 
-void InsetCollapsable::insetMotionNotify(BufferView * bv,
-                                         int x, int y, int state)
+int InsetCollapsable::latex(Buffer const * buf, ostream & os,
+                           bool fragile, bool free_spc) const
 {
-       if (y > button_bottom_y) {
-               LyXFont font(LyXFont::ALL_SANE);
-               int yy = ascent(bv, font) + y -
-                   (ascent_collapsed() +
-                    descent_collapsed() +
-                    inset.ascent(bv, font));
-               inset.insetMotionNotify(bv, x, yy, state);
-       }
+       return inset.latex(buf, os, fragile, free_spc);
 }
 
 
-void InsetCollapsable::insetKeyPress(XKeyEvent * xke)
+int InsetCollapsable::ascii(Buffer const * buf, ostream & os, int ll) const
 {
-       inset.insetKeyPress(xke);
+       return inset.ascii(buf, os, ll);
 }
 
 
-int InsetCollapsable::latex(Buffer const * buf, ostream & os,
-                            bool fragile, bool free_spc) const
+int InsetCollapsable::linuxdoc(Buffer const * buf, ostream & os) const
 {
-       return inset.latex(buf, os, fragile, free_spc);
+       return inset.linuxdoc(buf, os);
 }
 
-#if 0
-int InsetCollapsable::getMaxWidth(BufferView * bv,
-                                  UpdatableInset const * in) const
-{
-#if 0
-       int const w = UpdatableInset::getMaxWidth(bv, in);
 
-       if (w < 0) {
-               // What does a negative max width signify? (Lgb)
-               // Use the max width of the draw-area (Jug)
-               return w;
-       }
-       // should be at least 30 pixels !!!
-       return max(30, w - width_collapsed());
-#else
-       return UpdatableInset::getMaxWidth(bv, in);
-#endif
+int InsetCollapsable::docbook(Buffer const * buf, ostream & os, bool mixcont) const
+{
+       return inset.docbook(buf, os, mixcont);
 }
-#endif
 
 
-void InsetCollapsable::update(BufferView * bv, LyXFont const & font,
-                              bool reinit)
+void InsetCollapsable::update(BufferView * bv, bool reinit)
 {
        if (in_update) {
                if (reinit && owner()) {
-                       owner()->update(bv, font, true);
+                       owner()->update(bv, true);
                }
                return;
        }
        in_update = true;
-       inset.update(bv, font, reinit);
+       inset.update(bv, reinit);
        if (reinit && owner()) {
-               owner()->update(bv, font, true);
+               owner()->update(bv, true);
        }
        in_update = false;
 }
 
 
-UpdatableInset::RESULT
-InsetCollapsable::localDispatch(BufferView * bv, kb_action action,
-                                string const & arg)
-{
-       UpdatableInset::RESULT result = inset.localDispatch(bv, action, arg);
-       if (result >= FINISHED)
-               bv->unlockInset(this);
-       first_after_edit = false;
-       return result;
+Inset::RESULT InsetCollapsable::localDispatch(FuncRequest const & cmd)
+{
+       //lyxerr << "InsetCollapsable::localDispatch: " << cmd.action << "\n";
+       BufferView * bv = cmd.view();
+       switch (cmd.action) {
+               case LFUN_INSET_EDIT: {
+                       if (!cmd.argument.empty()) {
+                               UpdatableInset::localDispatch(cmd);
+                               if (collapsed_) {
+                                       collapsed_ = false;
+                                       if (bv->lockInset(this)) {
+                                               inset.setUpdateStatus(bv, InsetText::FULL);
+                                               bv->updateInset(this);
+                                               bv->buffer()->markDirty();
+                                               inset.localDispatch(cmd);
+                                               first_after_edit = true;
+                                       }
+                               } else {
+                                       if (bv->lockInset(this))
+                                               inset.localDispatch(cmd);
+                               }
+                               return DISPATCHED;
+                       }
+
+#ifdef WITH_WARNINGS
+#warning Fix this properly in BufferView_pimpl::workAreaButtonRelease
+#endif
+                       if (cmd.button() == mouse_button::button3)
+                               return DISPATCHED;
+
+                       UpdatableInset::localDispatch(cmd);
+
+                       if (collapsed_) {
+                               collapsed_ = false;
+                               // set this only here as it should be recollapsed only if
+                               // it was already collapsed!
+                               first_after_edit = true;
+                               if (!bv->lockInset(this))
+                                       return DISPATCHED;
+                               bv->updateInset(this);
+                               bv->buffer()->markDirty();
+                               inset.localDispatch(cmd);
+                       } else {
+                               FuncRequest cmd1 = cmd;
+                               if (!bv->lockInset(this))
+                                       return DISPATCHED;
+                               if (cmd.y <= button_bottom_y) {
+                                       cmd1.y = 0;
+                               } else {
+                                       LyXFont font(LyXFont::ALL_SANE);
+                                       cmd1.y = ascent(bv, font) + cmd.y -
+                                               (ascent_collapsed() +
+                                               descent_collapsed() +
+                                               inset.ascent(bv, font));
+                               }
+                               inset.localDispatch(cmd);
+                       }
+                       return DISPATCHED;
+               }
+
+               case LFUN_MOUSE_PRESS:
+                       if (!collapsed_ && cmd.y > button_bottom_y)
+                               inset.localDispatch(adjustCommand(cmd));
+                       return DISPATCHED;
+
+               case LFUN_MOUSE_MOTION:
+                       if (!collapsed_ && cmd.y > button_bottom_y)
+                               inset.localDispatch(adjustCommand(cmd));
+                       return DISPATCHED;
+
+               case LFUN_MOUSE_RELEASE:
+                       lfunMouseRelease(cmd);
+                       return DISPATCHED;
+
+               default:
+                       UpdatableInset::RESULT result = inset.localDispatch(cmd);
+                       if (result >= FINISHED)
+                               bv->unlockInset(this);
+                       first_after_edit = false;
+                       return result;
+       }
 }
 
 
@@ -441,7 +436,7 @@ bool InsetCollapsable::lockInsetInInset(BufferView * bv, UpdatableInset * in)
 
 
 bool InsetCollapsable::unlockInsetInInset(BufferView * bv, UpdatableInset * in,
-                                          bool lr)
+                                         bool lr)
 {
        if (&inset == in) {
                bv->unlockInset(this);
@@ -459,7 +454,7 @@ bool InsetCollapsable::updateInsetInInset(BufferView * bv, Inset *in)
 }
 
 
-unsigned int InsetCollapsable::insetInInsetY()
+int InsetCollapsable::insetInInsetY() const
 {
        return inset.insetInInsetY() - (top_baseline - inset.y());
 }
@@ -471,27 +466,15 @@ void InsetCollapsable::validate(LaTeXFeatures & features) const
 }
 
 
-void InsetCollapsable::getCursorPos(BufferView * bv, int & x, int & y) const
-{
-       inset.getCursorPos(bv, x , y);
-}
-
-
-void InsetCollapsable::toggleInsetCursor(BufferView * bv)
-{
-       inset.toggleInsetCursor(bv);
-}
-
-
-void InsetCollapsable::showInsetCursor(BufferView * bv, bool show)
+void InsetCollapsable::getCursor(BufferView & bv, int & x, int & y) const
 {
-       inset.showInsetCursor(bv, show);
+       inset.getCursor(bv, x, y);
 }
 
 
-void InsetCollapsable::hideInsetCursor(BufferView * bv)
+void InsetCollapsable::getCursorPos(BufferView * bv, int & x, int & y) const
 {
-       inset.hideInsetCursor(bv);
+       inset.getCursorPos(bv, x , y);
 }
 
 
@@ -513,18 +496,12 @@ UpdatableInset * InsetCollapsable::getFirstLockingInsetOfType(Inset::Code c)
 
 
 void InsetCollapsable::setFont(BufferView * bv, LyXFont const & font,
-                               bool toggleall, bool selectall)
+                              bool toggleall, bool selectall)
 {
        inset.setFont(bv, font, toggleall, selectall);
 }
 
 
-bool InsetCollapsable::doClearArea() const
-{
-       return inset.doClearArea();
-}
-
-
 LyXText * InsetCollapsable::getLyXText(BufferView const * bv,
                                       bool const recursive) const
 {
@@ -546,7 +523,7 @@ void InsetCollapsable::resizeLyXText(BufferView * bv, bool force) const
 }
 
 
-std::vector<string> const InsetCollapsable::getLabelList() const
+vector<string> const InsetCollapsable::getLabelList() const
 {
        return inset.getLabelList();
 }
@@ -557,7 +534,7 @@ bool InsetCollapsable::nodraw() const
        return inset.nodraw();
 }
 
+
 int InsetCollapsable::scroll(bool recursive) const
 {
        int sx = UpdatableInset::scroll(false);
@@ -569,22 +546,9 @@ int InsetCollapsable::scroll(bool recursive) const
 }
 
 
-Paragraph * InsetCollapsable::getParFromID(int id) const
-{
-       lyxerr[Debug::INFO] << "Looking for paragraph " << id << endl;
-       return inset.getParFromID(id);
-}
-
-
-Paragraph * InsetCollapsable::firstParagraph() const
-{
-       return inset.firstParagraph();
-}
-
-
-Paragraph * InsetCollapsable::getFirstParagraph(int i) const
+ParagraphList * InsetCollapsable::getParagraphs(int i) const
 {
-       return inset.getFirstParagraph(i);
+       return inset.getParagraphs(i);
 }
 
 
@@ -605,9 +569,9 @@ Inset * InsetCollapsable::getInsetFromID(int id_arg) const
 void InsetCollapsable::open(BufferView * bv)
 {
        if (!collapsed_) return;
-       
+
        collapsed_ = false;
-       bv->updateInset(this, false);
+       bv->updateInset(this);
 }
 
 
@@ -615,9 +579,9 @@ void InsetCollapsable::close(BufferView * bv) const
 {
        if (collapsed_)
                return;
-       
+
        collapsed_ = true;
-       bv->updateInset(const_cast<InsetCollapsable *>(this), false);
+       bv->updateInset(const_cast<InsetCollapsable *>(this));
 }
 
 
@@ -627,34 +591,72 @@ void InsetCollapsable::setLabel(string const & l) const
 }
 
 
+void InsetCollapsable::markErased()
+{
+       inset.markErased();
+}
+
+
+bool InsetCollapsable::nextChange(BufferView * bv, lyx::pos_type & length)
+{
+       bool found = inset.nextChange(bv, length);
+
+       if (first_after_edit && !found)
+               close(bv);
+       else if (!found)
+               first_after_edit = false;
+       return found;
+}
+
+
 bool InsetCollapsable::searchForward(BufferView * bv, string const & str,
-                                     bool const & cs, bool const & mw)
+                                    bool cs, bool mw)
 {
        bool found = inset.searchForward(bv, str, cs, mw);
        if (first_after_edit && !found)
                close(bv);
-       first_after_edit = false;
+       else if (!found)
+               first_after_edit = false;
        return found;
 }
 
 
 bool InsetCollapsable::searchBackward(BufferView * bv, string const & str,
-                                      bool const & cs, bool const & mw)
+                                     bool cs, bool mw)
 {
        bool found = inset.searchBackward(bv, str, cs, mw);
        if (first_after_edit && !found)
                close(bv);
-       first_after_edit = false;
+       else if (!found)
+               first_after_edit = false;
        return found;
 }
 
 
-string const InsetCollapsable::selectNextWordToSpellcheck(BufferView * bv,
-                                             float & value) const
+WordLangTuple const
+InsetCollapsable::selectNextWordToSpellcheck(BufferView * bv, float & value) const
 {
-       string const str = inset.selectNextWordToSpellcheck(bv, value);
-       if (first_after_edit && str.empty())
+       WordLangTuple word = inset.selectNextWordToSpellcheck(bv, value);
+       if (first_after_edit && word.word().empty())
                close(bv);
        first_after_edit = false;
-       return str;
+       return word;
+}
+
+
+void InsetCollapsable::addPreview(grfx::PreviewLoader & loader) const
+{
+       inset.addPreview(loader);
+}
+
+
+void InsetCollapsable::cache(BufferView * bv) const
+{
+       view_ = bv->owner()->view();
+}
+
+
+BufferView * InsetCollapsable::view() const
+{
+       return view_.lock().get();
 }