Merge pull request #20 from github/connect-without-tone

Update skin tone only when tone attribute exists

Author: dgraham

index.js

@@ -35,7 +35,10 @@ class GEmojiElement extends HTMLElement {
       image.src = this.getAttribute('fallback-src') || ''
       this.appendChild(image)
     }
-    updateTone(this)
+
+    if (this.hasAttribute('tone')) {
+      updateTone(this)
+    }
   }
 
   static get observedAttributes(): Array<string> {

test/test.js

@@ -29,6 +29,14 @@ describe('g-emoji', function() {
       assert.equal('G-EMOJI', el.nodeName)
       assert.equal('👋🏻', el.textContent)
     })
+
+    it('does not change skin tone when connected without tone attribute', function() {
+      const el = new window.GEmojiElement()
+      el.textContent = '👋🏻'
+      document.body.append(el)
+      assert.equal('G-EMOJI', el.nodeName)
+      assert.equal('👋🏻', el.textContent)
+    })
   })
 
   describe('in emoji-supporting platforms', function() {