Prevent JS exception in Toast's global event handler in case of missing key

[tzimmermann]

Since we've upgraded our app from @ui5/webcomponents@1.16.0 to 1.20.0, we've been getting some errors logged in our sentry that look like this:

We are not able to reproduce it yet, but the current assumption is that some browser quirk (like this one) or some auto-fill feature generates KeyboardEvents with undefined as key sometimes.

Thus, it would be nice to add some optional chaining for the key field here, in order to prevent logging noise.
Especially since this is a global event handler (we are not even using any Toast component in our app) that gets called a lot.

ui5-webcomponents/packages/main/src/Toast.ts

Line 25 in c99d0ec

const isMKey = e.key.toLowerCase() === "m";

[niyap]

Hello @SAP/ui5-webcomponents-topic-rl,

Please check the handleGlobalKeydown method of ui5-toast web component. On android mobile device, there are cases when e.key returns undefined so e.key.toLowerCase() throws an error.

Could you please and enhance the condition?

Thank you in advance!

Kind Regards,
Niya