Merge pull request #61 from github/fix-option-not-found

Fix `indicateDefaultOption` erroring when no options are visible

Author: jfuchs

src/index.ts

@@ -77,7 +77,7 @@ export default class Combobox {
     if (this.defaultFirstOption) {
       Array.from(this.list.querySelectorAll<HTMLElement>('[role="option"]:not([aria-disabled="true"])'))
         .filter(visible)[0]
-        .setAttribute('data-combobox-option-default', 'true')
+        ?.setAttribute('data-combobox-option-default', 'true')
     }
   }
 

test/test.js

@@ -288,5 +288,12 @@ describe('combobox-nav', function () {
       combobox.clearSelection()
       assert.equal(document.querySelectorAll('[data-combobox-option-default]').length, 1)
     })
+
+    it('does not error when no options are visible', () => {
+      assert.doesNotThrow(() => {
+        document.getElementById('list-id').style.display = 'none'
+        combobox.clearSelection()
+      })
+    })
   })
 })