Merge pull request #125 from github/st-update-method-name-checker

Update method name checker to clear message

Author: srt32

docs/_guide/anti-patterns.md

@@ -142,14 +142,18 @@ When naming a method, you should avoid naming it something that already exists o
       const goodEl = document.querySelector('.js-methodname-shadow-good-input')
       const badEl = document.querySelector('.js-methodname-shadow-bad-input')
       const warnEl = document.querySelector('.js-methodname-shadow-warn-input')
+      if (name === '') {
+        goodEl.hidden = true
+        return
+      }
       let warning = warnings[name]
       if (name !== name.toLowerCase() && name.toLowerCase() in HTMLElement.prototype) {
         warning = `it is too similar to \`${name.toLowerCase()}\` which already exists`
       } else if (name.startsWith('on') && !(name in HTMLElement.prototype)) {
         warning = 'starting with `on` suggests a coupling between the event and the method (see below)'
       }
       goodEl.hidden = warning || (name in HTMLElement.prototype)
-      warnEl.hidden = !warning 
+      warnEl.hidden = !warning
       badEl.hidden = warning || !(name in HTMLElement.prototype)
       if (warning) {
         warnEl.querySelector('span').textContent = warning