X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FText2.cpp;h=8eb70f8e328c5e455c27935774f62788d135dce8;hb=7920a12065458b4106d893cafd41afc53efd1b3c;hp=67c626801290568d618eee491b6b4df2b3e9c5c3;hpb=a4fa1a8f7bcca8613cd0b11c49f7986c0e397082;p=lyx.git diff --git a/src/Text2.cpp b/src/Text2.cpp index 67c6268012..8eb70f8e32 100644 --- a/src/Text2.cpp +++ b/src/Text2.cpp @@ -22,7 +22,6 @@ #include "Text.h" -#include "Bidi.h" #include "Buffer.h" #include "buffer_funcs.h" #include "BufferList.h" @@ -43,17 +42,16 @@ #include "TextClass.h" #include "TextMetrics.h" -#include "insets/InsetCollapsable.h" +#include "insets/InsetCollapsible.h" #include "mathed/InsetMathHull.h" #include "support/lassert.h" #include "support/debug.h" #include "support/gettext.h" +#include "support/lyxalgo.h" #include "support/textutils.h" -#include - #include using namespace std; @@ -157,7 +155,7 @@ void Text::setInsetFont(BufferView const & bv, pit_type pit, pos_type pos, Font const & font) { Inset * const inset = pars_[pit].getInset(pos); - LASSERT(inset && inset->resetFontEdit(), /**/); + LASSERT(inset && inset->resetFontEdit(), return); CursorSlice::idx_type endidx = inset->nargs(); for (CursorSlice cs(*inset); cs.idx() != endidx; ++cs.idx()) { @@ -176,7 +174,7 @@ void Text::setInsetFont(BufferView const & bv, pit_type pit, void Text::setLayout(pit_type start, pit_type end, docstring const & layout) { - LASSERT(start != end, /**/); + LASSERT(start != end, return); Buffer const & buffer = owner_->buffer(); BufferParams const & bp = buffer.params(); @@ -194,12 +192,13 @@ void Text::setLayout(pit_type start, pit_type end, // set layout over selection and make a total rebreak of those paragraphs void Text::setLayout(Cursor & cur, docstring const & layout) { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); pit_type start = cur.selBegin().pit(); pit_type end = cur.selEnd().pit() + 1; cur.recordUndoSelection(); setLayout(start, end, layout); + cur.setCurrentFont(); cur.forceBufferUpdate(); } @@ -218,7 +217,7 @@ static bool changeDepthAllowed(Text::DEPTH_CHANGE type, bool Text::changeDepthAllowed(Cursor & cur, DEPTH_CHANGE type) const { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); // this happens when selecting several cells in tabular (bug 2630) if (cur.selBegin().idx() != cur.selEnd().idx()) return false; @@ -238,7 +237,7 @@ bool Text::changeDepthAllowed(Cursor & cur, DEPTH_CHANGE type) const void Text::changeDepth(Cursor & cur, DEPTH_CHANGE type) { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); pit_type const beg = cur.selBegin().pit(); pit_type const end = cur.selEnd().pit() + 1; cur.recordUndoSelection(); @@ -264,6 +263,11 @@ void Text::changeDepth(Cursor & cur, DEPTH_CHANGE type) void Text::setFont(Cursor & cur, Font const & font, bool toggleall) { LASSERT(this == cur.text(), return); + + // If there is a selection, record undo before the cursor font is changed. + if (cur.selection()) + cur.recordUndoSelection(); + // Set the current_font // Determine basis font FontInfo layoutfont; @@ -289,10 +293,9 @@ void Text::setFont(Cursor & cur, Font const & font, bool toggleall) return; // Ok, we have a selection. - cur.recordUndoSelection(); Font newfont = font; - if (toggleall) { + if (toggleall) { // Toggling behaves as follows: We check the first character of the // selection. If it's (say) got EMPH on, then we set to off; if off, // then to on. With families and the like, we set it to INHERIT, if @@ -300,30 +303,30 @@ void Text::setFont(Cursor & cur, Font const & font, bool toggleall) CursorSlice const & sl = cur.selBegin(); Text const & text = *sl.text(); Paragraph const & par = text.getPar(sl.pit()); - + // get font at the position Font oldfont = par.getFont(cur.bv().buffer().params(), sl.pos(), text.outerFont(sl.pit())); FontInfo const & oldfi = oldfont.fontInfo(); - + FontInfo & newfi = newfont.fontInfo(); - + FontFamily newfam = newfi.family(); if (newfam != INHERIT_FAMILY && newfam != IGNORE_FAMILY && newfam == oldfi.family()) newfi.setFamily(INHERIT_FAMILY); - + FontSeries newser = newfi.series(); if (newser == BOLD_SERIES && oldfi.series() == BOLD_SERIES) newfi.setSeries(INHERIT_SERIES); - + FontShape newshp = newfi.shape(); - if (newshp != INHERIT_SHAPE && newshp != IGNORE_SHAPE && + if (newshp != INHERIT_SHAPE && newshp != IGNORE_SHAPE && newshp == oldfi.shape()) newfi.setShape(INHERIT_SHAPE); ColorCode newcol = newfi.color(); - if (newcol != Color_none && newcol != Color_inherit + if (newcol != Color_none && newcol != Color_inherit && newcol != Color_ignore && newcol == oldfi.color()) newfi.setColor(Color_none); @@ -334,6 +337,8 @@ void Text::setFont(Cursor & cur, Font const & font, bool toggleall) newfi.setUnderbar(oldfi.underbar() == FONT_OFF ? FONT_ON : FONT_OFF); if (newfi.strikeout() == FONT_TOGGLE) newfi.setStrikeout(oldfi.strikeout() == FONT_OFF ? FONT_ON : FONT_OFF); + if (newfi.xout() == FONT_TOGGLE) + newfi.setXout(oldfi.xout() == FONT_OFF ? FONT_ON : FONT_OFF); if (newfi.uuline() == FONT_TOGGLE) newfi.setUuline(oldfi.uuline() == FONT_OFF ? FONT_ON : FONT_OFF); if (newfi.uwave() == FONT_TOGGLE) @@ -344,7 +349,7 @@ void Text::setFont(Cursor & cur, Font const & font, bool toggleall) newfi.setNumber(oldfi.number() == FONT_OFF ? FONT_ON : FONT_OFF); } - setFont(cur.bv(), cur.selectionBegin().top(), + setFont(cur.bv(), cur.selectionBegin().top(), cur.selectionEnd().top(), newfont); } @@ -374,7 +379,7 @@ void Text::setFont(BufferView const & bv, CursorSlice const & begin, Font f = tm.displayFont(pit, pos); f.update(font, language); setCharFont(pit, pos, f, tm.font_); - // font change may change language... + // font change may change language... // spell checker has to know that pars_[pit].requestSpellCheck(pos); } @@ -383,21 +388,21 @@ void Text::setFont(BufferView const & bv, CursorSlice const & begin, bool Text::cursorTop(Cursor & cur) { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); return setCursor(cur, 0, 0); } bool Text::cursorBottom(Cursor & cur) { - LASSERT(this == cur.text(), /**/); - return setCursor(cur, cur.lastpit(), boost::prior(paragraphs().end())->size()); + LBUFERR(this == cur.text()); + return setCursor(cur, cur.lastpit(), prev(paragraphs().end(), 1)->size()); } void Text::toggleFree(Cursor & cur, Font const & font, bool toggleall) { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); // If the mask is completely neutral, tell user if (font.fontInfo() == ignore_font && font.language() == ignore_language) { // Could only happen with user style @@ -429,7 +434,7 @@ void Text::toggleFree(Cursor & cur, Font const & font, bool toggleall) docstring Text::getStringToIndex(Cursor const & cur) { - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); if (cur.selection()) return cur.selectionAsString(false); @@ -446,7 +451,7 @@ docstring Text::getStringToIndex(Cursor const & cur) cur.message(_("Cannot index more than one paragraph!")); else return tmpcur.selectionAsString(false); - + return docstring(); } @@ -480,9 +485,9 @@ void Text::setLabelWidthStringToSequence(Cursor const & cur, } -void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) +void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) { - LASSERT(cur.text(), /**/); + LBUFERR(cur.text()); //FIXME UNICODE string const argument = to_utf8(arg); @@ -498,7 +503,7 @@ void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) // with same layout in a sequence. // Do this only once for a selected range of paragraphs // of the same layout and depth. - cur.recordUndo(); + c.recordUndo(); par.params().apply(params, par.layout()); if (par.getDepth() != priordepth || par.layout() != priorlayout) setLabelWidthStringToSequence(c, params.labelWidthString()); @@ -508,12 +513,9 @@ void Text::setParagraphs(Cursor & cur, docstring arg, bool merge) } -//FIXME This is a little redundant now, but it's probably worth keeping, -//especially if we're going to go away from using serialization internally -//quite so much. -void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) +void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) { - LASSERT(cur.text(), /**/); + LBUFERR(cur.text()); depth_type priordepth = -1; Layout priorlayout; @@ -539,58 +541,31 @@ void Text::setParagraphs(Cursor & cur, ParagraphParameters const & p) // this really should just insert the inset and not move the cursor. void Text::insertInset(Cursor & cur, Inset * inset) { - LASSERT(this == cur.text(), /**/); - LASSERT(inset, /**/); + LBUFERR(this == cur.text()); + LBUFERR(inset); cur.paragraph().insertInset(cur.pos(), inset, cur.current_font, - Change(cur.buffer()->params().trackChanges + Change(cur.buffer()->params().track_changes ? Change::INSERTED : Change::UNCHANGED)); } -bool Text::setCursor(Cursor & cur, pit_type par, pos_type pos, +bool Text::setCursor(Cursor & cur, pit_type pit, pos_type pos, bool setfont, bool boundary) { TextMetrics const & tm = cur.bv().textMetrics(this); - bool const update_needed = !tm.contains(par); + bool const update_needed = !tm.contains(pit); Cursor old = cur; - setCursorIntern(cur, par, pos, setfont, boundary); - // FIXME There is a chance that we'll miss a screen update here. - // If so, then do DEPM and then check if cur wants an update and - // go ahead and do it, if so. + setCursorIntern(cur, pit, pos, setfont, boundary); return cur.bv().checkDepm(cur, old) || update_needed; } -void Text::setCursor(CursorSlice & cur, pit_type par, pos_type pos) +void Text::setCursorIntern(Cursor & cur, pit_type pit, pos_type pos, + bool setfont, bool boundary) { - LASSERT(par != int(paragraphs().size()), /**/); - cur.pit() = par; - cur.pos() = pos; - - // now some strict checking - Paragraph & para = getPar(par); - - // None of these should happen, but we're scaredy-cats - if (pos < 0) { - lyxerr << "don't like -1" << endl; - LASSERT(false, /**/); - } - - if (pos > para.size()) { - lyxerr << "don't like 1, pos: " << pos - << " size: " << para.size() - << " par: " << par << endl; - LASSERT(false, /**/); - } -} - - -void Text::setCursorIntern(Cursor & cur, - pit_type par, pos_type pos, bool setfont, bool boundary) -{ - LASSERT(this == cur.text(), /**/); + LBUFERR(this == cur.text()); cur.boundary(boundary); - setCursor(cur.top(), par, pos); + cur.top().setPitPos(pit, pos); if (setfont) cur.setCurrentFont(); } @@ -598,8 +573,6 @@ void Text::setCursorIntern(Cursor & cur, bool Text::checkAndActivateInset(Cursor & cur, bool front) { - if (cur.selection()) - return false; if (front && cur.pos() == cur.lastpos()) return false; if (!front && cur.pos() == 0) @@ -607,6 +580,8 @@ bool Text::checkAndActivateInset(Cursor & cur, bool front) Inset * inset = front ? cur.nextInset() : cur.prevInset(); if (!inset || !inset->editable()) return false; + if (cur.selection() && cur.realAnchor().find(inset) == -1) + return false; /* * Apparently, when entering an inset we are expected to be positioned * *before* it in the containing paragraph, regardless of the direction @@ -617,14 +592,14 @@ bool Text::checkAndActivateInset(Cursor & cur, bool front) if (!front) --cur.pos(); inset->edit(cur, front); + cur.setCurrentFont(); + cur.boundary(false); return true; } bool Text::checkAndActivateInsetVisual(Cursor & cur, bool movingForward, bool movingLeft) { - if (cur.selection()) - return false; if (cur.pos() == -1) return false; if (cur.pos() == cur.lastpos()) @@ -633,8 +608,12 @@ bool Text::checkAndActivateInsetVisual(Cursor & cur, bool movingForward, bool mo Inset * inset = par.isInset(cur.pos()) ? par.getInset(cur.pos()) : 0; if (!inset || !inset->editable()) return false; - inset->edit(cur, movingForward, + if (cur.selection() && cur.realAnchor().find(inset) == -1) + return false; + inset->edit(cur, movingForward, movingLeft ? Inset::ENTRY_DIRECTION_RIGHT : Inset::ENTRY_DIRECTION_LEFT); + cur.setCurrentFont(); + cur.boundary(false); return true; } @@ -662,21 +641,28 @@ bool Text::cursorBackward(Cursor & cur) cur.textRow().pos() == cur.pos() && !cur.paragraph().isLineSeparator(cur.pos() - 1) && !cur.paragraph().isNewline(cur.pos() - 1) && + !cur.paragraph().isEnvSeparator(cur.pos() - 1) && !cur.paragraph().isSeparator(cur.pos() - 1)) { return setCursor(cur, cur.pit(), cur.pos(), true, true); } - + // go left and try to enter inset if (checkAndActivateInset(cur, false)) return false; - + // normal character left return setCursor(cur, cur.pit(), cur.pos() - 1, true, false); } // move to the previous paragraph or do nothing - if (cur.pit() > 0) - return setCursor(cur, cur.pit() - 1, getPar(cur.pit() - 1).size(), true, false); + if (cur.pit() > 0) { + Paragraph & par = getPar(cur.pit() - 1); + pos_type lastpos = par.size(); + if (lastpos > 0 && par.isEnvSeparator(lastpos - 1)) + return setCursor(cur, cur.pit() - 1, lastpos - 1, true, false); + else + return setCursor(cur, cur.pit() - 1, lastpos, true, false); + } return false; } @@ -689,7 +675,7 @@ bool Text::cursorVisLeft(Cursor & cur, bool skip_inset) cur = temp_cur; return false; } - return setCursor(cur, temp_cur.pit(), temp_cur.pos(), + return setCursor(cur, temp_cur.pit(), temp_cur.pos(), true, temp_cur.boundary()); } @@ -724,7 +710,7 @@ bool Text::cursorForward(Cursor & cur) if (cur.boundary() && !tm.isRTLBoundary(cur.pit(), cur.pos())) return setCursor(cur, cur.pit(), cur.pos(), true, false); - // next position is left of boundary, + // next position is left of boundary, // but go to next line for special cases like space, newline, linesep #if 0 // some effectless debug code to see the values in the debugger @@ -740,19 +726,26 @@ bool Text::cursorForward(Cursor & cur) bool sep2 = cur.paragraph().isSeparator(cur.pos()+1); } #endif - if (cur.textRow().endpos() == cur.pos() + 1 && - cur.textRow().endpos() != cur.lastpos() && - !cur.paragraph().isNewline(cur.pos()) && - !cur.paragraph().isLineSeparator(cur.pos()) && - !cur.paragraph().isSeparator(cur.pos())) { - return setCursor(cur, cur.pit(), cur.pos() + 1, true, true); + if (cur.textRow().endpos() == cur.pos() + 1) { + if (cur.paragraph().isEnvSeparator(cur.pos()) && + cur.pos() + 1 == cur.lastpos() && + cur.pit() != cur.lastpit()) { + // move to next paragraph + return setCursor(cur, cur.pit() + 1, 0, true, false); + } else if (cur.textRow().endpos() != cur.lastpos() && + !cur.paragraph().isNewline(cur.pos()) && + !cur.paragraph().isEnvSeparator(cur.pos()) && + !cur.paragraph().isLineSeparator(cur.pos()) && + !cur.paragraph().isSeparator(cur.pos())) { + return setCursor(cur, cur.pit(), cur.pos() + 1, true, true); + } } - + // in front of RTL boundary? Stay on this side of the boundary because: // ab|cDDEEFFghi -> abc|DDEEFFghi if (tm.isRTLBoundary(cur.pit(), cur.pos() + 1)) return setCursor(cur, cur.pit(), cur.pos() + 1, true, true); - + // move right return setCursor(cur, cur.pit(), cur.pos() + 1, true, false); } @@ -779,7 +772,13 @@ bool Text::cursorDownParagraph(Cursor & cur) { bool updated = false; if (cur.pit() != cur.lastpit()) - updated = setCursor(cur, cur.pit() + 1, 0); + if (lyxrc.mac_like_cursor_movement) + if (cur.pos() == cur.lastpos()) + updated = setCursor(cur, cur.pit() + 1, getPar(cur.pit() + 1).size()); + else + updated = setCursor(cur, cur.pit(), cur.lastpos()); + else + updated = setCursor(cur, cur.pit() + 1, 0); else updated = setCursor(cur, cur.pit(), cur.lastpos()); return updated; @@ -791,7 +790,7 @@ bool Text::cursorDownParagraph(Cursor & cur) void Text::fixCursorAfterDelete(CursorSlice & cur, CursorSlice const & where) { // Do nothing if cursor is not in the paragraph where the - // deletion occured, + // deletion occurred, if (cur.pit() != where.pit()) return; @@ -840,12 +839,12 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, if (&old.inset() == &cur[depth].inset()) break; - // Whether a common inset is found and whether the cursor is still in + // Whether a common inset is found and whether the cursor is still in // the same paragraph (possibly nested). bool const same_par = depth < cur.depth() && old.pit() == cur[depth].pit(); - bool const same_par_pos = depth == cur.depth() - 1 && same_par + bool const same_par_pos = depth == cur.depth() - 1 && same_par && old.pos() == cur[depth].pos(); - + // If the chars around the old cursor were spaces, delete one of them. if (!same_par_pos) { // Only if the cursor has really moved. @@ -855,7 +854,7 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, && oldpar.isLineSeparator(old.pos() - 1) && !oldpar.isDeleted(old.pos() - 1) && !oldpar.isDeleted(old.pos())) { - oldpar.eraseChar(old.pos() - 1, cur.buffer()->params().trackChanges); + oldpar.eraseChar(old.pos() - 1, cur.buffer()->params().track_changes); // FIXME: This will not work anymore when we have multiple views of the same buffer // In this case, we will have to correct also the cursors held by // other bufferviews. It will probably be easier to do that in a more @@ -883,17 +882,16 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, if (oldpar.empty() || (oldpar.size() == 1 && oldpar.isLineSeparator(0))) { // Delete old par. - old.recordUndo(ATOMIC_UNDO, - max(old.pit() - 1, pit_type(0)), - min(old.pit() + 1, old.lastpit())); + old.recordUndo(max(old.pit() - 1, pit_type(0)), + min(old.pit() + 1, old.lastpit())); ParagraphList & plist = old.text()->paragraphs(); bool const soa = oldpar.params().startOfAppendix(); - plist.erase(boost::next(plist.begin(), old.pit())); + plist.erase(lyx::next(plist.begin(), old.pit())); // do not lose start of appendix marker (bug 4212) if (soa && old.pit() < pit_type(plist.size())) plist[old.pit()].params().startOfAppendix(true); - // see #warning (FIXME?) above + // see #warning (FIXME?) above if (cur.depth() >= old.depth()) { CursorSlice & curslice = cur[old.depth() - 1]; if (&curslice.inset() == &old.inset() @@ -910,7 +908,7 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, return true; } - if (oldpar.stripLeadingSpaces(cur.buffer()->params().trackChanges)) { + if (oldpar.stripLeadingSpaces(cur.buffer()->params().track_changes)) { need_anchor_change = true; // We return true here because the Paragraph contents changed and // we need a redraw before further action is processed. @@ -923,7 +921,7 @@ bool Text::deleteEmptyParagraphMechanism(Cursor & cur, void Text::deleteEmptyParagraphMechanism(pit_type first, pit_type last, bool trackChanges) { - LASSERT(first >= 0 && first <= last && last < (int) pars_.size(), /**/); + LASSERT(first >= 0 && first <= last && last < (int) pars_.size(), return); for (pit_type pit = first; pit <= last; ++pit) { Paragraph & par = pars_[pit]; @@ -952,7 +950,7 @@ void Text::deleteEmptyParagraphMechanism(pit_type first, pit_type last, bool tra continue; if (par.empty() || (par.size() == 1 && par.isLineSeparator(0))) { - pars_.erase(boost::next(pars_.begin(), pit)); + pars_.erase(lyx::next(pars_.begin(), pit)); --pit; --last; continue; @@ -963,15 +961,4 @@ void Text::deleteEmptyParagraphMechanism(pit_type first, pit_type last, bool tra } -void Text::recUndo(Cursor & cur, pit_type first, pit_type last) const -{ - cur.recordUndo(ATOMIC_UNDO, first, last); -} - - -void Text::recUndo(Cursor & cur, pit_type par) const -{ - cur.recordUndo(ATOMIC_UNDO, par, par); -} - } // namespace lyx