* make actionbar support different trigger keys

* add some undefined checks

We compile using the alwaysStrict flag so these directives are not needed

fixes #56648

* several accessibility fixes

* add checked item aria tag

* more cleanup

* add ts type

* restructure the menu and actionbar
ref #55258

* removing isMenu

remove hover theming for menus as it no longer applies
remove unnecessary and conflicting high contrast theme border
use border over outline like hc

* start

* SelectBox: Add select aria-label option

* Add aria-label to native select

* Revert debug action changes

* Cleanup

* submenu registration, ignore && mnemonic rendering for now

* clean up ActionItems for Menu Widget and submenu

* clean up the update actions logic in menubar

* fix keyboard navigation for submenus

* add terminal menu and update submenu labels

* fix missing separator issue

* updating some aria labels

* bring back action runner to hide menu before running

* address feedback from @bpasero

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'`.