From: Jean-Marc Lasgouttes <lasgouttes@lyx.org>
Date: Fri, 27 Jan 2017 15:09:03 +0000 (+0100)
Subject: Fix flushing of row that was cut after an hyphen
X-Git-Tag: 2.2.3~70
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=1438123a1316e06218b43ea7430a59ef747c506b;p=features.git

Fix flushing of row that was cut after an hyphen

When using Qt stuff in breakAt, it may happen that the row is broken
after an hyphen (whereas the old code would only consider spaces).

The fact that we abuse the Row::right_boundary() property to detect when
a row should be flushed broke justification when a row is cut at an
hyphen.

Fix this by introducing a new Row::flushed() property and set it as needed.
(cherry picked from commit 8e7d0c2002bdc69c95f3a43f7c78d13fe47ce5f3)
---

diff --git a/src/Row.cpp b/src/Row.cpp
index ba1a5aa9bb..f19a40bf60 100644
--- a/src/Row.cpp
+++ b/src/Row.cpp
@@ -148,7 +148,7 @@ Row::Row()
 	  sel_beg(-1), sel_end(-1),
 	  begin_margin_sel(false), end_margin_sel(false),
 	  changed_(false), crc_(0),
-	  pit_(0), pos_(0), end_(0), right_boundary_(false)
+	  pit_(0), pos_(0), end_(0), right_boundary_(false), flushed_(false)
 {}
 
 
diff --git a/src/Row.h b/src/Row.h
index b35c976083..91f0e73239 100644
--- a/src/Row.h
+++ b/src/Row.h
@@ -160,6 +160,10 @@ public:
 	void right_boundary(bool b) { right_boundary_ = b; }
 	///
 	bool right_boundary() const { return right_boundary_; }
+	///
+	void flushed(bool b) { flushed_ = b; }
+	///
+	bool flushed() const { return flushed_; }
 
 	///
 	Dimension const & dimension() const { return dim_; }
@@ -297,8 +301,10 @@ private:
 	pos_type pos_;
 	/// one behind last pos covered by this row
 	pos_type end_;
-	// Is there is a boundary at the end of the row (display inset...)
+	// Is there a boundary at the end of the row (display inset...)
 	bool right_boundary_;
+	// Shall the row be flushed when it is supposed to be justified?
+	bool flushed_;
 	/// Row dimension.
 	Dimension dim_;
 };
diff --git a/src/TextMetrics.cpp b/src/TextMetrics.cpp
index dbe1f6370d..280c90a3df 100644
--- a/src/TextMetrics.cpp
+++ b/src/TextMetrics.cpp
@@ -553,8 +553,7 @@ LyXAlignment TextMetrics::getAlign(Paragraph const & par, Row const & row) const
 		// not justify stuff, then don't stretch.
 		// A forced block alignment can only be overridden the 'no
 		// justification on screen' setting.
-		if (((row.right_boundary() || row.endpos() == par.size())
-		     && !forced_block)
+		if ((row.flushed() && !forced_block)
 		    || !bv_->buffer().params().justification)
 			align = text_->isRTL(par) ? LYX_ALIGN_RIGHT : LYX_ALIGN_LEFT;
 	}
@@ -904,7 +903,7 @@ void TextMetrics::breakRow(Row & row, int const right_margin, pit_type const pit
 			&& inset->display())
 		    || (!row.empty() && row.back().inset
 			&& row.back().inset->display())) {
-			row.right_boundary(true);
+			row.flushed(true);
 			need_new_row = par.isNewline(i);
 			++i;
 			break;
@@ -935,8 +934,13 @@ void TextMetrics::breakRow(Row & row, int const right_margin, pit_type const pit
 	// of success, reset indication that the row was broken abruptly.
 	int const next_width = max_width_ - leftMargin(max_width_, row.pit(), row.endpos())
 		- rightMargin(row.pit());
-	if (row.shortenIfNeeded(body_pos, width, next_width))
-		row.right_boundary(!row.empty() && row.back().endpos == row.endpos());
+
+	row.shortenIfNeeded(body_pos, width, next_width);
+	row.right_boundary(!row.empty() && row.endpos() < end
+	                   && row.back().endpos == row.endpos());
+	// Last row in paragraph is flushed
+	if (row.endpos() == end)
+		row.flushed(true);
 
 	// make sure that the RTL elements are in reverse ordering
 	row.reverseRTL(is_rtl);
diff --git a/status.22x b/status.22x
index 23deeb493e..8328b234b6 100644
--- a/status.22x
+++ b/status.22x
@@ -164,6 +164,8 @@ What's new
 - Fix wrong on-screen text layout where a word would be alone on a
   line.
 
+- Fix wrong on-screen flushing of a row that was cut after an hyphen.
+
 - Fix crash when the document contains Unicode line-breaking characters.
 
 - Allow using colors supported by xcolor inside mathed (bug 10417).