Fixing regressions text metrics, InputWidget

`run_Font_` was moving the Y cursor position twice for each line break.

Checking for the HarfBuzz UNSAFE_TO_BREAK flag leads to some unexpected behavior near edges of words.

The old `tryAdvanceNoWrap` method should return the maximum horizontal advance of the text, and not the cursor position's advance.

`draw_WrapText` used the wrong foreground color.

`TextBuf` now uses WrapText to do all the measuring and drawing, making things much simpler.

1 files changed, 4 insertions, 0 deletions

diff --git a/src/ui/text.h b/src/ui/text.h
index 09d92ce0..d779dd96 100644
--- a/src/ui/text.h
+++ b/src/ui/text.h
@@ -159,6 +159,10 @@ struct Impl_TextMetrics {
     iInt2 advance; /* cursor offset */
 };
 
+iLocalDef int maxWidth_TextMetrics(const iTextMetrics d) {
+    return iMax(width_Rect(d.bounds), d.advance.x);
+}
+
 iTextMetrics    measureRange_Text       (int fontId, iRangecc text);
 iTextMetrics    measureWrapRange_Text   (int fontId, int maxWidth, iRangecc text);
 iTextMetrics    measureN_Text           (int fontId, const char *text, size_t n); /* `n` in characters */