X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FText3.cpp;h=4c99c15774707dfe432fed950f1eae836d5185d4;hb=ecdeffb52b1dad2f56e0e4210d0a040e807e2caf;hp=cdc1bd488bc4d0c45f7fe2ae55ccd534050f0b07;hpb=3e8bfd8bd6e2ce6c1293cfe178a847a79ea32d49;p=lyx.git

diff --git a/src/Text3.cpp b/src/Text3.cpp
index cdc1bd488b..4c99c15774 100644
--- a/src/Text3.cpp
+++ b/src/Text3.cpp
@@ -73,6 +73,7 @@
 #include "support/lstrings.h"
 #include "support/lyxtime.h"
 #include "support/os.h"
+#include "support/regex.h"
 
 #include "mathed/InsetMathHull.h"
 #include "mathed/MathMacroTemplate.h"
@@ -148,7 +149,7 @@ static void mathDispatch(Cursor & cur, FuncRequest const & cmd, bool display)
 #endif
 		cur.insert(new InsetMathHull(cur.buffer(), hullSimple));
 #ifdef ENABLE_ASSERTIONS
-		LASSERT(old_pos == cur.pos(), /**/);
+		LATTEST(old_pos == cur.pos());
 #endif
 		cur.nextInset()->edit(cur, true);
 		// don't do that also for LFUN_MATH_MODE
@@ -344,8 +345,6 @@ static void outline(OutlineOp mode, Cursor & cur)
 	ParagraphList::iterator finish = start;
 	ParagraphList::iterator const end = pars.end();
 
-	DocumentClass const & tc = buf.params().documentClass();
-
 	int const thistoclevel = buf.text().getTocLevel(distance(bgn, start));
 	int toclevel;
 
@@ -408,18 +407,25 @@ static void outline(OutlineOp mode, Cursor & cur)
 			cur.pit() = newpit - len;
 			break;
 		}
-		case OutlineIn: {
+		case OutlineIn:
+		case OutlineOut: {
 			pit_type const len = distance(start, finish);
 			buf.undo().recordUndo(cur, ATOMIC_UNDO, pit, pit + len - 1);
 			for (; start != finish; ++start) {
 				toclevel = buf.text().getTocLevel(distance(bgn, start));
 				if (toclevel == Layout::NOT_IN_TOC)
 					continue;
+
+				DocumentClass const & tc = buf.params().documentClass();
 				DocumentClass::const_iterator lit = tc.begin();
 				DocumentClass::const_iterator len = tc.end();
+				int const newtoclevel = 
+					(mode == OutlineIn ? toclevel + 1 : toclevel - 1);
+				LabelType const oldlabeltype = start->layout().labeltype;
+
 				for (; lit != len; ++lit) {
-					if (lit->toclevel == toclevel + 1 &&
-					    start->layout().labeltype == lit->labeltype) {
+					if (lit->toclevel ==  newtoclevel &&
+					     lit->labeltype == oldlabeltype) {
 						start->setLayout(*lit);
 						break;
 					}
@@ -427,25 +433,6 @@ static void outline(OutlineOp mode, Cursor & cur)
 			}
 			break;
 		}
-		case OutlineOut: {
-			pit_type const len = distance(start, finish);
-			buf.undo().recordUndo(cur, ATOMIC_UNDO, pit, pit + len - 1);
-			for (; start != finish; ++start) {
-				toclevel = buf.text().getTocLevel(distance(bgn, start));
-				if (toclevel == Layout::NOT_IN_TOC)
-					continue;
-				DocumentClass::const_iterator lit = tc.begin();
-				DocumentClass::const_iterator len = tc.end();
-				for (; lit != len; ++lit) {
-					if (lit->toclevel == toclevel - 1 &&
-						start->layout().labeltype == lit->labeltype) {
-							start->setLayout(*lit);
-							break;
-					}
-				}
-			}
-			break;
-		}
 	}
 }
 
@@ -488,7 +475,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	// at the end?
 	cur.noScreenUpdate();
 
-	LASSERT(cur.text() == this, /**/);
+	LBUFERR(this == cur.text());
 	CursorSlice const oldTopSlice = cur.top();
 	bool const oldBoundary = cur.boundary();
 	bool const oldSelection = cur.selection();
@@ -501,7 +488,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		LyXAction::NoUpdate) || singleParUpdate);
 	bool const last_misspelled = lyxrc.spellcheck_continuously
 		&& cur.paragraph().isMisspelled(cur.pos(), true);
-	
+
 	FuncCode const act = cmd.action();
 	switch (act) {
 
@@ -567,7 +554,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		finishChange(cur, false);
 		break;
 
-	case LFUN_LINE_DELETE:
+	case LFUN_LINE_DELETE_FORWARD:
 		if (cur.selection())
 			cutSelection(cur, true, false);
 		else
@@ -615,18 +602,6 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		cur.screenUpdateFlags(Update::FitCursor);
 		break;
 
-	case LFUN_INSET_SELECT_ALL:
-		if (cur.depth() == 1 || !cur.selection() || !cur.selBegin().at_begin()
-			  || !cur.selEnd().at_end()) {
-			needsUpdate |= cur.selHandle(false);
-			needsUpdate |= cursorTop(cur);
-			needsUpdate |= cur.selHandle(true);
-			needsUpdate |= cursorBottom(cur);
-		} else 
-			cur.undispatched();
-		cur.screenUpdateFlags(Update::FitCursor);
-		break;
-
 	case LFUN_CHAR_FORWARD:
 	case LFUN_CHAR_FORWARD_SELECT:
 		//LYXERR0(" LFUN_CHAR_FORWARD[SEL]:\n" << cur);
@@ -637,23 +612,19 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 				&& cur.boundary() == oldBoundary) {
 			cur.undispatched();
 			cmd = FuncRequest(LFUN_FINISHED_FORWARD);
-		
+
 			// we will probably be moving out the inset, so we should execute
-			// the depm-mechanism, but only when the cursor has a place to 
+			// the depm-mechanism, but only when the cursor has a place to
 			// go outside this inset, i.e. in a slice above.
-			if (cur.depth() > 1 && cur.pos() == cur.lastpos() 
+			if (cur.depth() > 1 && cur.pos() == cur.lastpos()
 				  && cur.pit() == cur.lastpit()) {
-				// The cursor hasn't changed yet. To give the 
+				// The cursor hasn't changed yet. To give the
 				// DEPM the possibility of doing something we must
 				// provide it with two different cursors.
 				Cursor dummy = cur;
 				dummy.pos() = dummy.pit() = 0;
-				if (cur.bv().checkDepm(dummy, cur)) {
+				if (cur.bv().checkDepm(dummy, cur))
 					cur.forceBufferUpdate();
-					// DEPM may have requested a screen update
-					cur.screenUpdateFlags(
-						cur.screenUpdate() | dummy.screenUpdate());
-				}
 			}
 		}
 		break;
@@ -670,21 +641,17 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 			cmd = FuncRequest(LFUN_FINISHED_BACKWARD);
 
 			// we will probably be moving out the inset, so we should execute
-			// the depm-mechanism, but only when the cursor has a place to 
+			// the depm-mechanism, but only when the cursor has a place to
 			// go outside this inset, i.e. in a slice above.
 			if (cur.depth() > 1 && cur.pos() == 0 && cur.pit() == 0) {
-				// The cursor hasn't changed yet. To give the 
+				// The cursor hasn't changed yet. To give the
 				// DEPM the possibility of doing something we must
 				// provide it with two different cursors.
 				Cursor dummy = cur;
 				dummy.pos() = cur.lastpos();
 				dummy.pit() = cur.lastpit();
-				if (cur.bv().checkDepm(dummy, cur)) {
+				if (cur.bv().checkDepm(dummy, cur))
 					cur.forceBufferUpdate();
-					// DEPM may have requested a screen update
-					cur.screenUpdateFlags(
-						cur.screenUpdate() | dummy.screenUpdate());
-				}
 			}
 		}
 		break;
@@ -749,15 +716,14 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		bool const atFirstOrLastRow = cur.atFirstOrLastRow(up);
 
 		if (!atFirstOrLastRow) {
-			needsUpdate |= cur.selHandle(select);	
-			cur.selHandle(select);
+			needsUpdate |= cur.selHandle(select);
 			cur.upDownInText(up, needsUpdate);
 			needsUpdate |= cur.beforeDispatchCursor().inMathed();
 		} else {
 			// if the cursor cannot be moved up or down do not remove
 			// the selection right now, but wait for the next dispatch.
 			if (select)
-				needsUpdate |= cur.selHandle(select);	
+				needsUpdate |= cur.selHandle(select);
 			cur.upDownInText(up, needsUpdate);
 			cur.undispatched();
 		}
@@ -819,7 +785,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 				break;
 		}
 		cur.pos() = cur.lastpos();
-		
+
 		needsUpdate |= cur != old_cur;
 		break;
 	}
@@ -856,23 +822,19 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 				&& cur.boundary() == oldBoundary) {
 			cur.undispatched();
 			cmd = FuncRequest(LFUN_FINISHED_FORWARD);
-		
+
 			// we will probably be moving out the inset, so we should execute
-			// the depm-mechanism, but only when the cursor has a place to 
+			// the depm-mechanism, but only when the cursor has a place to
 			// go outside this inset, i.e. in a slice above.
-			if (cur.depth() > 1 && cur.pos() == cur.lastpos() 
+			if (cur.depth() > 1 && cur.pos() == cur.lastpos()
 				  && cur.pit() == cur.lastpit()) {
-				// The cursor hasn't changed yet. To give the 
+				// The cursor hasn't changed yet. To give the
 				// DEPM the possibility of doing something we must
 				// provide it with two different cursors.
 				Cursor dummy = cur;
 				dummy.pos() = dummy.pit() = 0;
-				if (cur.bv().checkDepm(dummy, cur)) {
+				if (cur.bv().checkDepm(dummy, cur))
 					cur.forceBufferUpdate();
-					// DEPM may have requested a screen update
-					cur.screenUpdateFlags(
-						cur.screenUpdate() | dummy.screenUpdate());
-				}
 			}
 		}
 		break;
@@ -904,29 +866,25 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	case LFUN_WORD_BACKWARD_SELECT:
 		needsUpdate |= cur.selHandle(cmd.action() == LFUN_WORD_BACKWARD_SELECT);
 		needsUpdate |= cursorBackwardOneWord(cur);
-	
+
 		if (!needsUpdate && oldTopSlice == cur.top()
 				&& cur.boundary() == oldBoundary) {
 			cur.undispatched();
 			cmd = FuncRequest(LFUN_FINISHED_BACKWARD);
-		
+
 			// we will probably be moving out the inset, so we should execute
-			// the depm-mechanism, but only when the cursor has a place to 
+			// the depm-mechanism, but only when the cursor has a place to
 			// go outside this inset, i.e. in a slice above.
-			if (cur.depth() > 1 && cur.pos() == 0 
+			if (cur.depth() > 1 && cur.pos() == 0
 				  && cur.pit() == 0) {
-				// The cursor hasn't changed yet. To give the 
+				// The cursor hasn't changed yet. To give the
 				// DEPM the possibility of doing something we must
 				// provide it with two different cursors.
 				Cursor dummy = cur;
 				dummy.pos() = cur.lastpos();
 				dummy.pit() = cur.lastpit();
-				if (cur.bv().checkDepm(dummy, cur)) {
+				if (cur.bv().checkDepm(dummy, cur))
 					cur.forceBufferUpdate();
-					// DEPM may have requested a screen update
-					cur.screenUpdateFlags(
-						cur.screenUpdate() | dummy.screenUpdate());
-				}
 			}
 		}
 		break;
@@ -940,15 +898,12 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	case LFUN_NEWLINE_INSERT: {
 		InsetNewlineParams inp;
 		docstring arg = cmd.argument();
-		// this avoids a double undo
-		// FIXME: should not be needed, ideally
-		if (!cur.selection())
-			cur.recordUndo();
-		cap::replaceSelection(cur);
 		if (arg == "linebreak")
 			inp.kind = InsetNewlineParams::LINEBREAK;
 		else
 			inp.kind = InsetNewlineParams::NEWLINE;
+		cap::replaceSelection(cur);
+		cur.recordUndo();
 		cur.insert(new InsetNewline(inp));
 		cur.posForward();
 		moveCursor(cur, false);
@@ -964,8 +919,8 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 			cur.recordUndoSelection();
 			pit_type const pit_end = cur.selEnd().pit();
 			for (pit_type pit = cur.selBegin().pit(); pit <= pit_end; pit++) {
-				pars_[pit].insertChar(0, '\t', 
-						      bv->buffer().params().trackChanges);
+				pars_[pit].insertChar(0, '\t',
+						      bv->buffer().params().track_changes);
 				// Update the selection pos to make sure the selection does not
 				// change as the inserted tab will increase the logical pos.
 				if (cur.realAnchor().pit() == pit)
@@ -978,13 +933,13 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 			// Maybe we shouldn't allow tabs within a line, because they
 			// are not (yet) aligned as one might do expect.
 			FuncRequest cmd(LFUN_SELF_INSERT, from_ascii("\t"));
-			dispatch(cur, cmd);	
+			dispatch(cur, cmd);
 		}
 		break;
 	}
 
 	case LFUN_TAB_DELETE: {
-		bool const tc = bv->buffer().params().trackChanges;
+		bool const tc = bv->buffer().params().track_changes;
 		if (cur.selection()) {
 			// If there is a selection, a tab (if present) is removed from
 			// the beginning of each paragraph.
@@ -998,11 +953,11 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 				if (c == '\t' || c == ' ') {
 					// remove either 1 tab or 4 spaces.
 					int const n = (c == ' ' ? 4 : 1);
-					for (int i = 0; i < n 
+					for (int i = 0; i < n
 						  && !par.empty() && par.getChar(0) == c; ++i) {
 						if (cur.pit() == pit)
 							cur.posBackward();
-						if (cur.realAnchor().pit() == pit 
+						if (cur.realAnchor().pit() == pit
 							  && cur.realAnchor().pos() > 0 )
 							cur.realAnchor().backwardPos();
 						par.eraseChar(0, tc);
@@ -1011,23 +966,23 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 			}
 			cur.finishUndo();
 		} else {
-			// If there is no selection, try to remove a tab or some spaces 
+			// If there is no selection, try to remove a tab or some spaces
 			// before the position of the cursor.
 			Paragraph & par = paragraphs()[cur.pit()];
 			pos_type const pos = cur.pos();
-			
+
 			if (pos == 0)
 				break;
-			
+
 			char_type const c = par.getChar(pos - 1);
 			cur.recordUndo();
 			if (c == '\t') {
 				cur.posBackward();
 				par.eraseChar(cur.pos(), tc);
 			} else
-				for (int n_spaces = 0; 
+				for (int n_spaces = 0;
 				     cur.pos() > 0
-					     && par.getChar(cur.pos() - 1) == ' ' 
+					     && par.getChar(cur.pos() - 1) == ' '
 					     && n_spaces < 4;
 				     ++n_spaces) {
 					cur.posBackward();
@@ -1040,11 +995,20 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 
 	case LFUN_CHAR_DELETE_FORWARD:
 		if (!cur.selection()) {
+			bool was_separator = cur.paragraph().isEnvSeparator(cur.pos());
 			if (cur.pos() == cur.paragraph().size())
 				// Par boundary, force full-screen update
 				singleParUpdate = false;
 			needsUpdate |= erase(cur);
 			cur.resetAnchor();
+			if (was_separator && cur.pos() == cur.paragraph().size()
+			    && (!cur.paragraph().layout().isEnvironment()
+				|| cur.paragraph().size() > 0)) {
+				// Force full-screen update
+				singleParUpdate = false;
+				needsUpdate |= erase(cur);
+				cur.resetAnchor();
+			}
 			// It is possible to make it a lot faster still
 			// just comment out the line below...
 		} else {
@@ -1057,11 +1021,17 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	case LFUN_CHAR_DELETE_BACKWARD:
 		if (!cur.selection()) {
 			if (bv->getIntl().getTransManager().backspace()) {
+				bool par_boundary = cur.pos() == 0;
 				// Par boundary, full-screen update
-				if (cur.pos() == 0)
+				if (par_boundary)
 					singleParUpdate = false;
 				needsUpdate |= backspace(cur);
 				cur.resetAnchor();
+				if (par_boundary && cur.pos() > 0
+				    && cur.paragraph().isEnvSeparator(cur.pos() - 1)) {
+					needsUpdate |= backspace(cur);
+					cur.resetAnchor();
+				}
 				// It is possible to make it a lot faster still
 				// just comment out the line below...
 			}
@@ -1071,11 +1041,42 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		}
 		break;
 
-	case LFUN_PARAGRAPH_BREAK:
+	case LFUN_PARAGRAPH_BREAK: {
 		cap::replaceSelection(cur);
-		breakParagraph(cur, cmd.argument() == "inverse");
+		pit_type pit = cur.pit();
+		Paragraph const & par = pars_[pit];
+		pit_type prev = pit;
+		if (pit > 0) {
+			if (!pars_[pit - 1].layout().isEnvironment())
+				prev = depthHook(pit, par.getDepth());
+			else if (pars_[pit - 1].getDepth() >= par.getDepth())
+				prev = pit - 1;
+		}
+		if (prev < pit && cur.pos() == par.beginOfBody()
+		    && !par.isEnvSeparator(cur.pos())
+		    && !par.layout().isCommand()
+		    && pars_[prev].layout() != par.layout()
+		    && pars_[prev].layout().isEnvironment()) {
+			if (par.layout().isEnvironment()
+			    && pars_[prev].getDepth() == par.getDepth()) {
+				docstring const layout = par.layout().name();
+				DocumentClass const & tc = bv->buffer().params().documentClass();
+				lyx::dispatch(FuncRequest(LFUN_LAYOUT, tc.plainLayout().name()));
+				lyx::dispatch(FuncRequest(LFUN_SEPARATOR_INSERT, "parbreak"));
+				lyx::dispatch(FuncRequest(LFUN_PARAGRAPH_BREAK, "inverse"));
+				lyx::dispatch(FuncRequest(LFUN_LAYOUT, layout));
+			} else {
+				lyx::dispatch(FuncRequest(LFUN_SEPARATOR_INSERT, "parbreak"));
+				breakParagraph(cur);
+			}
+			Font const f(inherit_font, cur.current_font.language());
+			pars_[cur.pit() - 1].resetFonts(f);
+		} else {
+			breakParagraph(cur, cmd.argument() == "inverse");
+		}
 		cur.resetAnchor();
 		break;
+	}
 
 	case LFUN_INSET_INSERT: {
 		cur.recordUndo();
@@ -1100,7 +1101,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 			/*
 			Paragraph & par = pars_[cur.pit()];
 			if (inset->lyxCode() == LABEL_CODE
-				&& par.layout().labeltype == LABEL_COUNTER) {
+				&& !par.layout().counter.empty() {
 				// Go to the end of the paragraph
 				// Warning: Because of Change-Tracking, the last
 				// position is 'size()' and not 'size()-1':
@@ -1239,23 +1240,30 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 
 	case LFUN_PASTE: {
 		cur.message(_("Paste"));
-		LASSERT(cur.selBegin().idx() == cur.selEnd().idx(), /**/);
+		LASSERT(cur.selBegin().idx() == cur.selEnd().idx(), break);
 		cap::replaceSelection(cur);
 
 		// without argument?
 		string const arg = to_utf8(cmd.argument());
 		if (arg.empty()) {
+			bool tryGraphics = true;
 			if (theClipboard().isInternal())
 				pasteFromStack(cur, bv->buffer().errorList("Paste"), 0);
-			else if (theClipboard().hasGraphicsContents() 
-				     && !theClipboard().hasTextContents())
+			else if (theClipboard().hasTextContents()) {
+				if (pasteClipboardText(cur, bv->buffer().errorList("Paste"),
+				                       true, Clipboard::AnyTextType))
+					tryGraphics = false;
+			}
+			if (tryGraphics && theClipboard().hasGraphicsContents())
 				pasteClipboardGraphics(cur, bv->buffer().errorList("Paste"));
-			else
-				pasteClipboardText(cur, bv->buffer().errorList("Paste"));
 		} else if (isStrUnsignedInt(arg)) {
 			// we have a numerical argument
 			pasteFromStack(cur, bv->buffer().errorList("Paste"),
 				       convert<unsigned int>(arg));
+		} else if (arg == "html" || arg == "latex") {
+			Clipboard::TextType type = (arg == "html") ?
+				Clipboard::HtmlTextType : Clipboard::LaTeXTextType;
+			pasteClipboardText(cur, bv->buffer().errorList("Paste"), true, type);
 		} else {
 			Clipboard::GraphicsType type = Clipboard::AnyGraphicsType;
 			if (arg == "pdf")
@@ -1270,9 +1278,10 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 				type = Clipboard::EmfGraphicsType;
 			else if (arg == "wmf")
 				type = Clipboard::WmfGraphicsType;
-
 			else
-				LASSERT(false, /**/);
+				// We used to assert, but couldn't the argument come from, say, the
+				// minibuffer and just be mistyped?
+				LYXERR0("Unrecognized graphics type: " << arg);
 
 			pasteClipboardGraphics(cur, bv->buffer().errorList("Paste"), type);
 		}
@@ -1389,23 +1398,37 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	}
 
 	case LFUN_ENVIRONMENT_SPLIT: {
+		bool const outer = cmd.argument() == "outer";
 		Paragraph const & para = cur.paragraph();
-		docstring const layout = para.layout().name();
-		if (cur.pos() > 0) {
-			FuncRequest cmd(LFUN_PARAGRAPH_BREAK);
-			lyx::dispatch(cmd);
+		docstring layout = para.layout().name();
+		depth_type split_depth = cur.paragraph().params().depth();
+		if (outer) {
+			// check if we have an environment in our nesting hierarchy
+			pit_type pit = cur.pit();
+			Paragraph cpar = pars_[pit];
+			while (true) {
+				if (pit == 0 || cpar.params().depth() == 0)
+					break;
+				--pit;
+				cpar = pars_[pit];
+				if (cpar.params().depth() < split_depth
+				    && cpar.layout().isEnvironment()) {
+						layout = cpar.layout().name();
+						split_depth = cpar.params().depth();
+				}
+			}
 		}
-		bool const morecont = cur.lastpos() > cur.pos();
-		FuncRequest cmd2(LFUN_LAYOUT, "Separator");
-		lyx::dispatch(cmd2);
-		FuncRequest cmd3(LFUN_PARAGRAPH_BREAK, "inverse");
-		lyx::dispatch(cmd3);
-		if (morecont) {
-			FuncRequest cmd4(LFUN_DOWN);
-			lyx::dispatch(cmd4);
+		if (cur.pos() > 0)
+			lyx::dispatch(FuncRequest(LFUN_PARAGRAPH_BREAK));
+		if (outer) {
+			while (cur.paragraph().params().depth() > split_depth)
+				lyx::dispatch(FuncRequest(LFUN_DEPTH_DECREMENT));
 		}
-		FuncRequest cmd5(LFUN_LAYOUT, layout);
-		lyx::dispatch(cmd5);
+		DocumentClass const & tc = bv->buffer().params().documentClass();
+		lyx::dispatch(FuncRequest(LFUN_LAYOUT, tc.plainLayout().name()));
+		lyx::dispatch(FuncRequest(LFUN_SEPARATOR_INSERT, "plain"));
+		lyx::dispatch(FuncRequest(LFUN_PARAGRAPH_BREAK, "inverse"));
+		lyx::dispatch(FuncRequest(LFUN_LAYOUT, layout));
 
 		break;
 	}
@@ -1454,25 +1477,25 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 	}
 
 	case LFUN_QUOTE_INSERT: {
-		// this avoids a double undo
-		// FIXME: should not be needed, ideally
-		if (!cur.selection())
-			cur.recordUndo();
 		cap::replaceSelection(cur);
+		cur.recordUndo();
 
 		Paragraph const & par = cur.paragraph();
 		pos_type pos = cur.pos();
+		// Ignore deleted text before cursor
+		while (pos > 0 && par.isDeleted(pos - 1))
+			--pos;
 
 		BufferParams const & bufparams = bv->buffer().params();
-		bool const hebrew = 
+		bool const hebrew =
 			par.getFontSettings(bufparams, pos).language()->lang() == "hebrew";
 		bool const allow_inset_quote = !(par.isPassThru() || hebrew);
-		
+
+		string const arg = to_utf8(cmd.argument());
 		if (allow_inset_quote) {
 			char_type c = ' ';
 			if (pos > 0 && (!cur.prevInset() || !cur.prevInset()->isSpace()))
 				c = par.getChar(pos - 1);
-			string const arg = to_utf8(cmd.argument());
 			InsetQuotes::QuoteTimes const quote_type = (arg == "single")
 				? InsetQuotes::SingleQuotes : InsetQuotes::DoubleQuotes;
 			cur.insert(new InsetQuotes(cur.buffer(), c, quote_type));
@@ -1480,8 +1503,9 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		} else {
 			// The cursor might have been invalidated by the replaceSelection.
 			cur.buffer()->changed(true);
-			lyx::dispatch(FuncRequest(LFUN_SELF_INSERT, "\""));
-		}			
+			string const quote_string = (arg == "single") ? "'" : "\"";
+			lyx::dispatch(FuncRequest(LFUN_SELF_INSERT, quote_string));
+		}
 		break;
 	}
 
@@ -1505,7 +1529,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 
 	case LFUN_MOUSE_DOUBLE:
 		if (cmd.button() == mouse_button::button1) {
-			selectWord(cur, WHOLE_WORD_STRICT);
+			selectWord(cur, WHOLE_WORD);
 			bv->cursor() = cur;
 		}
 		break;
@@ -1663,16 +1687,47 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		break;
 	}
 
-	case LFUN_HYPERLINK_INSERT: {
-		InsetCommandParams p(HYPERLINK_CODE);
-		docstring content;
+	case LFUN_HREF_INSERT: {
+		// FIXME If we're actually given an argument, shouldn't
+		// we use it, whether or not we have a selection?
+		docstring content = cmd.argument();
 		if (cur.selection()) {
 			content = cur.selectionAsString(false);
 			cutSelection(cur, true, false);
 		}
-		p["target"] = (cmd.argument().empty()) ?
-			content : cmd.argument();
+
+		InsetCommandParams p(HYPERLINK_CODE);
+		if (!content.empty()){
+			// if it looks like a link, we'll put it as target,
+			// otherwise as name (bug #8792).
+
+			// We can't do:
+			//   regex_match(to_utf8(content), matches, link_re)
+			// because smatch stores pointers to the substrings rather
+			// than making copies of them. And those pointers become
+			// invalid after regex_match returns, since it is then
+			// being given a temporary object. (Thanks to Georg for
+			// figuring that out.)
+			regex const link_re("^([a-z]+):.*");
+			smatch matches;
+			string const c = to_utf8(lowercase(content));
+
+			if (c.substr(0,7) == "mailto:") {
+				p["target"] = content;
+				p["type"] = from_ascii("mailto:");
+			} else if (regex_match(c, matches, link_re)) {
+				p["target"] = content;
+				string protocol = matches.str(1);
+				if (protocol == "file")
+					p["type"] = from_ascii("file:");
+			} else
+				p["name"] = content;
+		}
 		string const data = InsetCommand::params2string(p);
+
+		// we need to have a target. if we already have one, then
+		// that gets used at the default for the name, too, which
+		// is probably what is wanted.
 		if (p["target"].empty()) {
 			bv->showDialog("href", data);
 		} else {
@@ -1855,7 +1910,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		dispatch(cur, fr);
 		break;
 	}
-	
+
 	case LFUN_NOMENCL_PRINT:
 	case LFUN_NEWPAGE_INSERT:
 		// do nothing fancy
@@ -1863,6 +1918,16 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		cur.posForward();
 		break;
 
+	case LFUN_SEPARATOR_INSERT: {
+		doInsertInset(cur, this, cmd, false, false);
+		cur.posForward();
+		// remove a following space
+		Paragraph & par = cur.paragraph();
+		if (cur.pos() != cur.lastpos() && par.isLineSeparator(cur.pos()))
+		    par.eraseChar(cur.pos(), cur.buffer()->params().track_changes);
+		break;
+	}
+
 	case LFUN_DEPTH_DECREMENT:
 		changeDepth(cur, DEC_DEPTH);
 		break;
@@ -1931,7 +1996,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		cap::replaceSelection(cur);
 		cur.insert(new InsetMathHull(cur.buffer(), hullSimple));
 		checkAndActivateInset(cur, true);
-		LASSERT(cur.inMathed(), /**/);
+		LASSERT(cur.inMathed(), break);
 		cur.dispatch(cmd);
 		break;
 	}
@@ -1999,14 +2064,14 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		break;
 	}
 
-	case LFUN_FONT_UULINE: {
+	case LFUN_FONT_UNDERUNDERLINE: {
 		Font font(ignore_font, ignore_language);
 		font.fontInfo().setUuline(FONT_TOGGLE);
 		toggleAndShow(cur, this, font);
 		break;
 	}
 
-	case LFUN_FONT_UWAVE: {
+	case LFUN_FONT_UNDERWAVE: {
 		Font font(ignore_font, ignore_language);
 		font.fontInfo().setUwave(FONT_TOGGLE);
 		toggleAndShow(cur, this, font);
@@ -2317,6 +2382,33 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 		needsUpdate = true;
 		break;
 
+	case LFUN_SERVER_GET_STATISTICS:
+		{
+			DocIterator from, to;
+			if (cur.selection()) {
+				from = cur.selectionBegin();
+				to = cur.selectionEnd();
+			} else {
+				from = doc_iterator_begin(cur.buffer());
+				to = doc_iterator_end(cur.buffer());
+			}
+
+			cur.buffer()->updateStatistics(from, to);
+			string const arg0 = cmd.getArg(0);
+			if (arg0 == "words") {
+				cur.message(convert<docstring>(cur.buffer()->wordCount()));
+			} else if (arg0 == "chars") {
+				cur.message(convert<docstring>(cur.buffer()->charCount(false)));
+			} else if (arg0 == "chars-space") {
+				cur.message(convert<docstring>(cur.buffer()->charCount(true)));
+			} else {
+				cur.message(convert<docstring>(cur.buffer()->wordCount()) + " "
+				+ convert<docstring>(cur.buffer()->charCount(false)) + " "
+				+ convert<docstring>(cur.buffer()->charCount(true)));
+			}
+		}
+		break;
+
 	default:
 		LYXERR(Debug::ACTION, "Command " << cmd << " not DISPATCHED by Text");
 		cur.undispatched();
@@ -2382,7 +2474,7 @@ void Text::dispatch(Cursor & cur, FuncRequest & cmd)
 bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 			FuncStatus & flag) const
 {
-	LASSERT(cur.text() == this, /**/);
+	LBUFERR(this == cur.text());
 
 	FontInfo const & fontinfo = cur.real_current_font.fontInfo();
 	bool enable = true;
@@ -2401,7 +2493,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 
 	case LFUN_APPENDIX:
 		// FIXME We really should not allow this to be put, e.g.,
-		// in a footnote, or in ERT. But it would make sense in a 
+		// in a footnote, or in ERT. But it would make sense in a
 		// branch, so I'm not sure what to do.
 		flag.setOnOff(cur.paragraph().params().startOfAppendix());
 		break;
@@ -2436,6 +2528,10 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 			code = INDEX_CODE;
 		else if (cmd.argument() == "index_print")
 			code = INDEX_PRINT_CODE;
+		else if (cmd.argument() == "listings")
+			code = LISTINGS_CODE;
+		else if (cmd.argument() == "mathspace")
+			code = MATH_HULL_CODE;
 		else if (cmd.argument() == "nomenclature")
 			code = NOMENCL_CODE;
 		else if (cmd.argument() == "nomencl_print")
@@ -2458,8 +2554,6 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 			code = VSPACE_CODE;
 		else if (cmd.argument() == "wrap")
 			code = WRAP_CODE;
-		else if (cmd.argument() == "listings")
-			code = LISTINGS_CODE;
 		break;
 
 	case LFUN_ERT_INSERT:
@@ -2535,11 +2629,17 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 		}
 		break;
 	}
-	case LFUN_CAPTION_INSERT:
+	case LFUN_CAPTION_INSERT: {
 		code = CAPTION_CODE;
-		// not allowed in description items
-		enable = !inDescriptionItem(cur);
+		string arg = cmd.getArg(0);
+		bool varia = arg != "LongTableNoNumber"
+			&& cur.inset().allowsCaptionVariation(arg);
+		// not allowed in description items,
+		// and in specific insets
+		enable = !inDescriptionItem(cur)
+			&& (varia || arg.empty() || arg == "Standard");
 		break;
+	}
 	case LFUN_NOTE_INSERT:
 		code = NOTE_CODE;
 		// in commands (sections etc.) and description items,
@@ -2654,7 +2754,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 		// not allowed in description items
 		enable = !inDescriptionItem(cur);
 		break;
-	case LFUN_HYPERLINK_INSERT:
+	case LFUN_HREF_INSERT:
 		if (cur.selIsMultiCell() || cur.selIsMultiLine()) {
 			enable = false;
 			break;
@@ -2773,6 +2873,20 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 			break;
 		}
 
+		// explicit text type?
+		if (arg == "html") {
+			// Do not enable for PlainTextType, since some tidying in the
+			// frontend is needed for HTML, which is too unsafe for plain text.
+			enable = theClipboard().hasTextContents(Clipboard::HtmlTextType);
+			break;
+		} else if (arg == "latex") {
+			// LaTeX is usually not available on the clipboard with
+			// the correct MIME type, but in plain text.
+			enable = theClipboard().hasTextContents(Clipboard::PlainTextType) ||
+			         theClipboard().hasTextContents(Clipboard::LaTeXTextType);
+			break;
+		}
+
 		// explicit graphics type?
 		Clipboard::GraphicsType type = Clipboard::AnyGraphicsType;
 		if ((arg == "pdf" && (type = Clipboard::PdfGraphicsType))
@@ -2837,7 +2951,13 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 
 	case LFUN_NEWLINE_INSERT:
 		// LaTeX restrictions (labels or empty par)
-		enable = (cur.pos() > cur.paragraph().beginOfBody());
+		enable = !cur.paragraph().isPassThru()
+			&& cur.pos() > cur.paragraph().beginOfBody();
+		break;
+
+	case LFUN_SEPARATOR_INSERT:
+		// Always enabled for now
+		enable = true;
 		break;
 
 	case LFUN_TAB_INSERT:
@@ -2875,7 +2995,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_PARAGRAPH_BREAK:
 		enable = cur.inset().getLayout().isMultiPar();
 		break;
-	
+
 	case LFUN_SPELLING_ADD:
 	case LFUN_SPELLING_IGNORE:
 	case LFUN_SPELLING_REMOVE:
@@ -2885,14 +3005,30 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_LAYOUT:
 		enable = !cur.inset().forcePlainLayout();
 		break;
-	
+
 	case LFUN_ENVIRONMENT_SPLIT: {
-		if (!cur.buffer()->params().documentClass().hasLayout(from_ascii("Separator"))
-		    || !cur.paragraph().layout().isEnvironment()) {
-			enable = false;
+		if (cmd.argument() == "outer") {
+			// check if we have an environment in our nesting hierarchy
+			bool res = false;
+			depth_type const current_depth = cur.paragraph().params().depth();
+			pit_type pit = cur.pit();
+			Paragraph cpar = pars_[pit];
+			while (true) {
+				if (pit == 0 || cpar.params().depth() == 0)
+					break;
+				--pit;
+				cpar = pars_[pit];
+				if (cpar.params().depth() < current_depth)
+					res = cpar.layout().isEnvironment();
+			}
+			enable = res;
 			break;
 		}
-		enable = true;
+		else if (cur.paragraph().layout().isEnvironment()) {
+			enable = true;
+			break;
+		}
+		enable = false;
 		break;
 	}
 
@@ -2926,8 +3062,8 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_FONT_STATE:
 	case LFUN_FONT_UNDERLINE:
 	case LFUN_FONT_STRIKEOUT:
-	case LFUN_FONT_UULINE:
-	case LFUN_FONT_UWAVE:
+	case LFUN_FONT_UNDERUNDERLINE:
+	case LFUN_FONT_UNDERWAVE:
 	case LFUN_TEXTSTYLE_APPLY:
 	case LFUN_TEXTSTYLE_UPDATE:
 		enable = !cur.paragraph().isPassThru();
@@ -2935,7 +3071,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 
 	case LFUN_WORD_DELETE_FORWARD:
 	case LFUN_WORD_DELETE_BACKWARD:
-	case LFUN_LINE_DELETE:
+	case LFUN_LINE_DELETE_FORWARD:
 	case LFUN_WORD_FORWARD:
 	case LFUN_WORD_BACKWARD:
 	case LFUN_WORD_RIGHT:
@@ -2970,7 +3106,6 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_INSET_END:
 	case LFUN_INSET_BEGIN_SELECT:
 	case LFUN_INSET_END_SELECT:
-	case LFUN_INSET_SELECT_ALL:
 	case LFUN_PARAGRAPH_UP:
 	case LFUN_PARAGRAPH_DOWN:
 	case LFUN_LINE_BEGIN:
@@ -2988,6 +3123,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_UNICODE_INSERT:
 	case LFUN_THESAURUS_ENTRY:
 	case LFUN_ESCAPE:
+	case LFUN_SERVER_GET_STATISTICS:
 		// these are handled in our dispatch()
 		enable = true;
 		break;
@@ -3010,7 +3146,7 @@ bool Text::getStatus(Cursor & cur, FuncRequest const & cmd,
 	}
 
 	if (code != NO_CODE
-	    && (cur.empty() 
+	    && (cur.empty()
 		|| !cur.inset().insetAllowed(code)
 		|| (cur.paragraph().layout().pass_thru && !allow_in_passthru)))
 		enable = false;