For the clear button it shouldn't matter whether there are
search results found or not. The clear button gets now enabled
after every search.

For #46315

Revert "Fix #47157 - Focus include/exclude box when toggling open"

This reverts commit cf87ba36.

See #46315

Revert "Fix #46628 - wrong "no results found" message"

This reverts commit 0bd949c0.

See #46315

Revert "Fix default search exclusions string"

This reverts commit 234d1006.

See #46315

Revert "Improve migrating from split include/exclude inputs to the new merged one"

This reverts commit 22f49f8c.

See #46315

Revert "Fix #45702 - merge include/exclude inputs"

This reverts commit 7fdf1e3b.

See #46315

fixes #46536

#45063

and include keyboard shortcut in label.
#45063

This reverts commit 27256832 which turns out did not break the import of vs/nls

This reverts commit 7de696a2.

Seems to have broken loading vs/nls in at least one case in insiders

Don't use this import form in places where it is not needed. This prevents us from correctly detecting duplicate imports

There are many places in the code that do `import Event, { ...} from '.../event'`.