Handle Chrome double escape behaviour

[nicodh]

resolves #4365

Pressing escape key twice will close open dialogs in Chrome no matter how onCancel is configured https://issues.chromium.org/issues/346597066

To handle that we move the onClickBackButton function one component higher to add it as onClose prop to the Dialog and handle the event the same way a BackButton Click would be handled

TODO: check other dialogs how the behave on forced close (should go into a new Issue though)

[nicodh]

Also tested with Instant Onboarding (closing the dialog when in profile creation process. (The rebuilded dialog looses the contactName but works after entering a profile name, so I left it as rare edge case)

[WofWca]

Does this MR handle all the canEscapeKeyClose={false} occurences?

And shouldn't we remove / rename that prop?

[nicodh]

Does this MR handle all the canEscapeKeyClose={false} occurences?

No it doesn't (as mentioned in the MR description)

should we rename the property

I would not, since once we have other browser with tauri they hopefully behave as expected

[WofWca]

OK, this improves things. I tested it with both no accounts and with other accounts present.

But I'd say we should keep that issue open, to handle other such dialogs, and to perhaps find a more thorough approach.

Because canEscapeKeyClose doesn't actually do what it advertises. So looks like we ought to always handle onClose or onCancel on all dialogs. But let's keep further discussions in #4365, or a follow-up issue, I suppose.