Changeset 31788

Timestamp:

03/16/2015 12:05:05 AM (10 years ago)

Author:

pento

Message:

Emoji: Move a comment to match the restructure in [31779], and add an extra comment explaining how emoji rendering support is detected.

See #31242

File:

• trunk/src/wp-includes/js/wp-emoji.js (modified) (2 diffs)

trunk/src/wp-includes/js/wp-emoji.js

@@ -83 +83 @@
             }
 
+            /*
+             * Chrome on OS X added native emoji rendering in M41. Unfortunately,
+             * it doesn't work when the font is bolder than 500 weight. So, we
+             * check for bold rendering support to avoid invisible emoji in Chrome.
+             */
             context.textBaseline = 'top';
             context.font = '600 32px Arial';
@@ -100 +105 @@
             } else {
                 /*
-                 * Chrome on OS X added native emoji rendering in M41. Unfortunately,
-                 * it doesn't work when the font is bolder than 500 weight. So, we
-                 * check for bold rendering support to avoid invisible emoji in Chrome.
+                 * This creates a smiling emoji, and checks to see if there is any image data in the
+                 * center pixel. In browsers that don't support emoji, the character will be rendered
+                 * as an empty square, so the center pixel will be blank.
                  */
                 context.fillText( String.fromCharCode( 55357, 56835 ), 0, 0 );