Ignore popup selection after disabling Combobox or Multiselect

packages/react-widgets/src/Combobox.tsx

@@ -288,6 +288,8 @@ const ComboboxImpl: Combobox = React.forwardRef(function Combobox<TDataItem>(
     data: string | TDataItem,
     originalEvent: React.SyntheticEvent,
   ) => {
+    if (isDisabled) return
+
     toggle.close()
     shouldFilter.current = false
 

packages/react-widgets/src/Multiselect.tsx

@@ -447,6 +447,8 @@ const Multiselect: Multiselect = React.forwardRef(function Multiselect<
     dataItem: TDataItem | undefined,
     originalEvent: React.SyntheticEvent,
   ) => {
+    if (isDisabled) return
+
     if (dataItem === undefined) return
 
     originalEvent.preventDefault()

packages/react-widgets/test/Combobox-test.js

@@ -10,6 +10,34 @@ describe('Combobox', function () {
     { label: 'suzy smith', id: 3 },
   ]
 
+  it('ignores option clicks after the open widget becomes disabled', () => {
+    const onChange = sinon.spy()
+    const onSelect = sinon.spy()
+    const widget = mount(
+      <Combobox
+        data={['Apple', 'Orange']}
+        defaultValue="Apple"
+        onChange={onChange}
+        onSelect={onSelect}
+        open
+      />,
+    )
+
+    expect(
+      widget
+        .find('[role="option"]')
+        .someWhere((option) => option.text() === 'Orange'),
+    ).to.equal(true)
+    widget.setProps({ disabled: true })
+    widget
+      .find('[role="option"]')
+      .filterWhere((option) => option.text() === 'Orange')
+      .simulate('click')
+
+    expect(onChange.called).to.equal(false)
+    expect(onSelect.called).to.equal(false)
+  })
+
   it('should set initial values', function () {
     mount(<Combobox value={'hello'} onChange={() => {}} />)
       .find('input.rw-input')

packages/react-widgets/test/Multiselect-test.js

@@ -11,6 +11,34 @@ describe('Multiselect', function () {
     { label: 'pat', id: 2 },
   ]
 
+  it('ignores option clicks after the open widget becomes disabled', () => {
+    const onChange = sinon.spy()
+    const onSelect = sinon.spy()
+    const widget = mount(
+      <Multiselect
+        data={['Apple', 'Orange']}
+        defaultValue={['Apple']}
+        onChange={onChange}
+        onSelect={onSelect}
+        open
+      />,
+    )
+
+    expect(
+      widget
+        .find('[role="option"]')
+        .someWhere((option) => option.text() === 'Orange'),
+    ).to.equal(true)
+    widget.setProps({ disabled: true })
+    widget
+      .find('[role="option"]')
+      .filterWhere((option) => option.text() === 'Orange')
+      .simulate('click')
+
+    expect(onChange.called).to.equal(false)
+    expect(onSelect.called).to.equal(false)
+  })
+
   it('should set initial values', function () {
     mount(<Multiselect value={['hello']} onChange={() => {}} />)
       .find(MultiselectTag)