From db8abfec04aa714879879b1d5104367e69cd96e8 Mon Sep 17 00:00:00 2001
From: Juergen Spitzmueller <spitz@lyx.org>
Date: Thu, 11 Jul 2013 11:37:49 +0200
Subject: [PATCH] Properly nest ulem commands within other font changes

This reverts the wrong fix to #8424 which caused #8733 and fixes both problems properly
---
 src/Font.cpp | 95 ++++++++++++++++++++++++----------------------------
 status.20x   |  3 ++
 2 files changed, 47 insertions(+), 51 deletions(-)

diff --git a/src/Font.cpp b/src/Font.cpp
index 76e0f2e161..3602e31020 100644
--- a/src/Font.cpp
+++ b/src/Font.cpp
@@ -393,30 +393,6 @@ int Font::latexWriteStartChanges(odocstream & os, BufferParams const & bparams,
 		count += 6;
 		env = true; //We have opened a new environment
 	}
-	if (f.underbar() == FONT_ON) {
-		os << "\\uline{";
-		count += 10;
-		runparams.inulemcmd = true;
-		env = true; //We have opened a new environment
-	}
-	if (f.strikeout() == FONT_ON) {
-		os << "\\sout{";
-		count += 9;
-		runparams.inulemcmd = true;
-		env = true; //We have opened a new environment
-	}
-	if (f.uuline() == FONT_ON) {
-		os << "\\uuline{";
-		count += 11;
-		runparams.inulemcmd = true;
-		env = true; //We have opened a new environment
-	}
-	if (f.uwave() == FONT_ON) {
-		os << "\\uwave{";
-		count += 10;
-		runparams.inulemcmd = true;
-		env = true; //We have opened a new environment
-	}
 	// \noun{} is a LyX special macro
 	if (f.noun() == FONT_ON) {
 		os << "\\noun{";
@@ -431,8 +407,31 @@ int Font::latexWriteStartChanges(odocstream & os, BufferParams const & bparams,
 		}
 		os << '\\'
 		   << LaTeXSizeNames[f.size()]
-		   << '{';
-		count += strlen(LaTeXSizeNames[f.size()]) + 2;
+		   << "{}";
+		count += strlen(LaTeXSizeNames[f.size()]) + 3;
+	}
+	// The ulem commands need to be on the deepest nesting level
+	// because ulem puts every nested group or macro in a box,
+	// which prevents linebreaks (#8424, #8733)
+	if (f.underbar() == FONT_ON) {
+		os << "\\uline{";
+		count += 10;
+		runparams.inulemcmd = true;
+	}
+	if (f.strikeout() == FONT_ON) {
+		os << "\\sout{";
+		count += 9;
+		runparams.inulemcmd = true;
+	}
+	if (f.uuline() == FONT_ON) {
+		os << "\\uuline{";
+		count += 11;
+		runparams.inulemcmd = true;
+	}
+	if (f.uwave() == FONT_ON) {
+		os << "\\uwave{";
+		count += 10;
+		runparams.inulemcmd = true;
 	}
 	return count;
 }
@@ -481,30 +480,6 @@ int Font::latexWriteEndChanges(otexstream & os, BufferParams const & bparams,
 		++count;
 		env = true; // Size change need not bother about closing env.
 	}
-	if (f.underbar() == FONT_ON) {
-		os << '}';
-		++count;
-		runparams.inulemcmd = false;
-		env = true; // Size change need not bother about closing env.
-	}
-	if (f.strikeout() == FONT_ON) {
-		os << '}';
-		++count;
-		runparams.inulemcmd = false;
-		env = true; // Size change need not bother about closing env.
-	}
-	if (f.uuline() == FONT_ON) {
-		os << '}';
-		++count;
-		runparams.inulemcmd = false;
-		env = true; // Size change need not bother about closing env.
-	}
-	if (f.uwave() == FONT_ON) {
-		os << '}';
-		++count;
-		runparams.inulemcmd = false;
-		env = true; // Size change need not bother about closing env.
-	}
 	if (f.noun() == FONT_ON) {
 		os << '}';
 		++count;
@@ -516,11 +491,29 @@ int Font::latexWriteEndChanges(otexstream & os, BufferParams const & bparams,
 			os << '}';
 			++count;
 		}
+	}
+	if (f.underbar() == FONT_ON) {
 		os << '}';
 		++count;
+		runparams.inulemcmd = false;
+	}
+	if (f.strikeout() == FONT_ON) {
+		os << '}';
+		++count;
+		runparams.inulemcmd = false;
+	}
+	if (f.uuline() == FONT_ON) {
+		os << '}';
+		++count;
+		runparams.inulemcmd = false;
+	}
+	if (f.uwave() == FONT_ON) {
+		os << '}';
+		++count;
+		runparams.inulemcmd = false;
 	}
 
-	// When the current language is Hebrew, Arabic, or Farsi
+	// If the current language is Hebrew, Arabic, or Farsi
 	// the numbers are written Left-to-Right. ArabTeX package
 	// reorders the number automatically but the packages used
 	// for Hebrew and Farsi (Arabi) do not.
diff --git a/status.20x b/status.20x
index beebe07301..ca26caf20f 100644
--- a/status.20x
+++ b/status.20x
@@ -52,6 +52,9 @@ What's new
 
 * DOCUMENT INPUT/OUTPUT
 
+- Fix output of ulem commands (underline etc.) which was broken as of
+  LyX 2.0.6 (bug 8733).
+
 - Fix problem with unbalanced braces in XeTeX output (bug 8765).
 
 - Added h5 and h6 as tags for Paragraph and Subparagraph.