fix(useHotKey): do not prevent hotkeys on hidden modal/dialog

[skjnldsv]

Found working on Server upgrade. If the viewer is idling waiting to be shown, the element is still present in the dom.
We need to ensure it's also really shown.

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable
• 2️⃣ Backport to stable8 for maintained Vue 2 version or not applicable

[skjnldsv]

I was a bit too quick, let me adjust the test

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.01%. Comparing base (e5737a2) to head (fb0e3d6).
⚠️ Report is 25 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7966      +/-   ##
==========================================
+ Coverage   52.99%   53.01%   +0.01%     
==========================================
  Files         100      100              
  Lines        3119     3120       +1     
  Branches      867      868       +1     
==========================================
+ Hits         1653     1654       +1     
  Misses       1228     1228              
  Partials      238      238              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[skjnldsv]

/backport to stable8

[ShGKme]

Fine for a hotfix, but I'd propose to remove allowInModal in favour of hotkey scoping similar to the focus trap.

So that:

• It doesn't depend on a CSS class in the modal
• It supports not only NcModal but other modal popovers as well like NcPopover
• It supports nesting, for example, when a file picker dialog is open from the settings dialog, and each has its own hotkeys

[skjnldsv]

Fine for a hotfix, but I'd propose to remove allowInModal in favour of hotkey scoping similar to the focus trap.

Referenced in new ticket 👍

[Antreesy]

Otherwise works fine, smoke tested in the app

[backportbot]

The backport to stable8 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable8
git pull origin stable8

# Create the new backport branch
git checkout -b backport/7966/stable8

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick fb0e3d68

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/7966/stable8

Error: Failed to check for changes with origin/stable8: No changes found in backport branch

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[skjnldsv]

/backport fb0e3d6 to stable8

[backportbot]

The backport to stable8 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable8
git pull origin stable8

# Create the new backport branch
git checkout -b backport/7966/stable8

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick fb0e3d6

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/7966/stable8

Error: Failed to check for changes with origin/stable8: No changes found in backport branch

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.