revert: Fixes unexpected autosuggest dropdown reopen when clicking outside browser window and back (#3316)

Author: just-boris

src/autosuggest/__tests__/autosuggest.test.tsx

@@ -8,7 +8,6 @@ import { KeyCode } from '@cloudscape-design/test-utils-core/utils';
 
 import '../../__a11y__/to-validate-a11y';
 import Autosuggest, { AutosuggestProps } from '../../../lib/components/autosuggest';
-import { documentHasFocus } from '../../../lib/components/internal/utils/dom';
 import createWrapper from '../../../lib/components/test-utils/dom';
 
 import itemStyles from '../../../lib/components/internal/components/selectable-item/styles.css.js';
@@ -58,15 +57,8 @@ jest.mock('@cloudscape-design/component-toolkit/internal', () => {
     warnOnce: jest.fn(),
   };
 });
-
-jest.mock('../../../lib/components/internal/utils/dom', () => ({
-  ...jest.requireActual('../../../lib/components/internal/utils/dom'),
-  documentHasFocus: jest.fn(() => true),
-}));
-
 beforeEach(() => {
-  (warnOnce as jest.Mock).mockClear();
-  (documentHasFocus as jest.Mock).mockClear();
+  (warnOnce as any).mockClear();
 });
 
 test('renders correct labels when focused', () => {
@@ -143,24 +135,21 @@ test('entered text option should not get screenreader override', () => {
   ).toBeFalsy();
 });
 
-test('should close dropdown on blur when document is in focus', () => {
-  const { wrapper } = renderAutosuggest(<Autosuggest enteredTextLabel={v => v} value="1" options={defaultOptions} />);
+test('should not close dropdown when no realted target in blur', () => {
+  const { wrapper, container } = renderAutosuggest(
+    <div>
+      <Autosuggest enteredTextLabel={v => v} value="1" options={defaultOptions} />
+      <button id="focus-target">focus target</button>
+    </div>
+  );
   wrapper.findNativeInput().focus();
   expect(wrapper.findDropdown().findOpenDropdown()).not.toBe(null);
 
-  wrapper.findNativeInput().blur();
-  expect(wrapper.findDropdown().findOpenDropdown()).toBe(null);
-});
-
-test('should not close dropdown on blur when document is not in focus', () => {
-  (documentHasFocus as jest.Mock).mockImplementation(() => false);
-
-  const { wrapper } = renderAutosuggest(<Autosuggest enteredTextLabel={v => v} value="1" options={defaultOptions} />);
-  wrapper.findNativeInput().focus();
+  document.body.focus();
   expect(wrapper.findDropdown().findOpenDropdown()).not.toBe(null);
 
-  wrapper.findNativeInput().blur();
-  expect(wrapper.findDropdown().findOpenDropdown()).not.toBe(null);
+  createWrapper(container).find('#focus-target')!.focus();
+  expect(wrapper.findDropdown().findOpenDropdown()).toBe(null);
 });
 
 it('should warn if recoveryText is provided without associated handler', () => {

src/internal/components/autosuggest-input/index.tsx

@@ -18,7 +18,6 @@ import { FormFieldValidationControlProps, useFormFieldContext } from '../../cont
 import { BaseKeyDetail, fireCancelableEvent, fireNonCancelableEvent, NonCancelableEventHandler } from '../../events';
 import { InternalBaseComponentProps } from '../../hooks/use-base-component';
 import { KeyCode } from '../../keycode';
-import { documentHasFocus } from '../../utils/dom';
 import { nodeBelongs } from '../../utils/node-belongs';
 import Dropdown from '../dropdown';
 import { ExpandToViewport } from '../dropdown/interfaces';
@@ -131,11 +130,6 @@ const AutosuggestInput = React.forwardRef(
     }));
 
     const handleBlur = () => {
-      // When the document in not in focus it means the focus moved outside the page. In that case the dropdown shall stay open
-      // so that when the user comes back it does not re-open unexpectedly when the focus is restored by the browser.
-      if (!documentHasFocus()) {
-        return;
-      }
       if (!preventCloseOnBlurRef.current) {
         closeDropdown();
         fireNonCancelableEvent(onBlur, null);

src/internal/utils/dom.ts

@@ -128,11 +128,3 @@ export function isSVGElement(target: unknown): target is SVGElement {
       typeof target.ownerSVGElement === 'object')
   );
 }
-
-export function documentHasFocus() {
-  // In JSDOM the hasFocus() always returns false which differs from the in-browser experience, see: https://github.com/jsdom/jsdom/issues/3794.
-  // Thus, when detecting JSDOM environment we return true here explicitly for the tests to work expectedly.
-  // The detection depends on the default userAgent set in JSDOM, see: https://github.com/jsdom/jsdom?tab=readme-ov-file#advanced-configuration.
-  const isJSDOM = typeof navigator !== 'undefined' && navigator.userAgent?.includes('jsdom');
-  return isJSDOM || (typeof document !== undefined && document.hasFocus());
-}