Merge pull request #349 from UnityTech/fix_emoji

Adjust emoji advance and size.

Author: zhuxingwei

Runtime/ui/txt/emoji.cs

@@ -233,9 +233,14 @@ public static Image image {
 
         public const int rowCount = 36;
         public const int colCount = 37;
+        public static float advanceFactor = 1.3f;
+        public static float sizeFactor = 1.2f;
 
         public static Rect getMinMaxRect(float fontSize, float ascent, float descent) {
-            return Rect.fromLTWH(fontSize * 0.05f, descent - fontSize, fontSize * 0.9f, fontSize * 0.9f);
+            return Rect.fromLTWH((advanceFactor - sizeFactor) / 2 * fontSize,
+                descent - fontSize * sizeFactor,
+                fontSize * sizeFactor,
+                fontSize * sizeFactor);
         }
 
         public static Rect getUVRect(int code) {

Runtime/ui/txt/layout.cs

@@ -8,7 +8,7 @@ public static float measureText(string text, TextStyle style) {
             char startingChar = text[0];
             float totalWidth = 0;
             if (char.IsHighSurrogate(startingChar) || EmojiUtils.isSingleCharEmoji(startingChar)) {
-                float advance = style.fontSize + style.letterSpacing;
+                float advance = style.fontSize * EmojiUtils.advanceFactor + style.letterSpacing;
                 for (int i = 0; i < text.Length; i++) {
                     char ch = text[i];
                     if (char.IsHighSurrogate(ch) || EmojiUtils.isSingleCharNonEmptyEmoji(ch)) {
@@ -42,7 +42,7 @@ public static int computeTruncateCount(float offset, string text, int start, int
             char startingChar = text[start];
             float currentAdvance = offset;
             if (char.IsHighSurrogate(startingChar) || EmojiUtils.isSingleCharEmoji(startingChar)) {
-                float advance = style.fontSize + style.letterSpacing;
+                float advance = style.fontSize * EmojiUtils.advanceFactor + style.letterSpacing;
                 for (int i = 0; i < count; i++) {
                     char ch = text[start + i];
                     if (char.IsHighSurrogate(ch) || EmojiUtils.isSingleCharNonEmptyEmoji(ch)) {
@@ -85,7 +85,7 @@ public static float computeCharWidths(float offset, string text, int start, int
             char startingChar = text[start];
             float totalWidths = 0;
             if (char.IsHighSurrogate(startingChar) || EmojiUtils.isSingleCharEmoji(startingChar)) {
-                float advance = style.fontSize + style.letterSpacing;
+                float advance = style.fontSize * EmojiUtils.advanceFactor + style.letterSpacing;
                 for (int i = 0; i < count; i++) {
                     char ch = text[start + i];
                     if (char.IsHighSurrogate(ch) || EmojiUtils.isSingleCharNonEmptyEmoji(ch)) {
@@ -224,16 +224,15 @@ static float _layoutEmoji(string text, int start, int count, TextStyle style, Fo
                     x += letterSpaceHalfLeft;
                     advances[i] = letterSpaceHalfLeft;
 
-
+                    float advance = style.fontSize * EmojiUtils.advanceFactor;
                     var minX = x;
-                    var maxX = metrics.descent - metrics.ascent + x;
-                    var minY = metrics.ascent;
+                    var maxX = advance + x;
+                    var minY = -style.fontSize * EmojiUtils.sizeFactor;
                     var maxY = metrics.descent;
                     _updateBounds(minX, maxX, minY, maxY, ref bounds);
 
                     positions[i] = x;
 
-                    float advance = style.fontSize;
                     x += advance;
 
                     advances[i] += advance;