STORM-276 FIXED Squiggles overflow line editor height when font size is set to large

- Also reduced squiggle width from 6 to 4 and prevented running past the end of a word

2 files changed, 18 insertions, 8 deletions

diff --git a/indra/llui/lllineeditor.cpp b/indra/llui/lllineeditor.cpp
index f119b0d9bc..d87b9d930c 100644
--- a/indra/llui/lllineeditor.cpp
+++ b/indra/llui/lllineeditor.cpp
@@ -1727,6 +1727,10 @@ void LLLineEditor::draw()
 	background.stretch( -mBorderThickness );
 
 	S32 lineeditor_v_pad = (background.getHeight() - mGLFont->getLineHeight()) / 2;
+	if (mSpellCheck)
+	{
+		lineeditor_v_pad += 1;
+	}
 
 	drawBackground();
 	
@@ -1945,12 +1949,15 @@ void LLLineEditor::draw()
 			if (pxEnd > pxWidth)
 				pxEnd = pxWidth;
 
+			S32 pxBottom = (S32)(text_bottom + mGLFont->getDescenderHeight());
+
 			gGL.color4ub(255, 0, 0, 200);
-			while (pxStart < pxEnd)
+			while (pxStart + 1 < pxEnd)
 			{
-				gl_line_2d(pxStart, (S32)text_bottom - 2, pxStart + 3, (S32)text_bottom + 1);
-				gl_line_2d(pxStart + 3, (S32)text_bottom + 1, pxStart + 6, (S32)text_bottom - 2);
-				pxStart += 6;
+				gl_line_2d(pxStart, pxBottom, pxStart + 2, pxBottom - 2);
+				if (pxStart + 3 < pxEnd)
+					gl_line_2d(pxStart + 2, pxBottom - 3, pxStart + 4, pxBottom - 1);
+				pxStart += 4;
 			}
 		}
 	}
diff --git a/indra/llui/lltextbase.cpp b/indra/llui/lltextbase.cpp
index 990c442b73..4db1efdd20 100644
--- a/indra/llui/lltextbase.cpp
+++ b/indra/llui/lltextbase.cpp
@@ -688,12 +688,15 @@ void LLTextBase::drawText()
 				squiggle_start += squiggle_end / 2 - pony * 3;
 				squiggle_end = squiggle_start + pony * 6;
 
+				S32 squiggle_bottom = text_rect.mBottom + (S32)cur_segment->getStyle()->getFont()->getDescenderHeight();
+
 				gGL.color4ub(255, 0, 0, 200);
-				while (squiggle_start < squiggle_end)
+				while (squiggle_start + 1 < squiggle_end)
 				{
-					gl_line_2d(squiggle_start, text_rect.mBottom - 2, squiggle_start + 3, text_rect.mBottom + 1);
-					gl_line_2d(squiggle_start + 3, text_rect.mBottom + 1, squiggle_start + 6, text_rect.mBottom - 2);
-					squiggle_start += 6;
+					gl_line_2d(squiggle_start, squiggle_bottom, squiggle_start + 2, squiggle_bottom - 2);
+					if (squiggle_start + 3 < squiggle_end)
+						gl_line_2d(squiggle_start + 2, squiggle_bottom - 3, squiggle_start + 4, squiggle_bottom - 1);
+					squiggle_start += 4;
 				}
 
 				if (misspell_it->second > seg_end)