fix(NcDialog): allow to close NcDialog on click outside

[Antreesy]

☑️ Resolves

• Follow-up from feat(NcDialog): New component (moved from @nextcloud/dialogs) #4550
• It feels counter-intuitive now for some dialogs (ex-modals) to not be closed with outside click. That doesn't apply to small confirmation popups, but some general things should be reverted to old behavior with this:
  • NcAppSettingsDialog
  • FilePicker
  • e.t.c.

🚧 Tasks

• Alternative: make closeOnClickOutside true by default ?

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable

[szaimen]

Makes sense

[susnux]

Alternative: make closeOnClickOutside true by default

I would say no for the filepicker we had this discussion somewhere. The close button is always visible - even if scrolled - on NcDialogs so users can explicitly close it. This prevents erroneous aborting just because of clicking a bit too far outside.

And I think it really depends on the content of your dialog, a simple confirmation might be dismissed by click outside, while something like a user creation dialog might lead to data loss and frustration just because of an accidental click (e.g. while trying to click "submit" or "accept" - so if the dialog backdrop is not inert we probably need more padding around buttons to allow all users to have frustration free UX).