fix(editor): support Cmd key for editor shortcuts on macOS

[xhon-pelushi]

Summary

• Register calendar editor shortcuts via @nextcloud/vue useHotKey so macOS uses Cmd and other platforms use Ctrl (not both).
• Call useHotKey directly from EditFull and EditSimple setup().

Fixes #8715

Test plan

• On macOS: open an event editor and confirm Cmd+Enter / Cmd+Delete / Cmd+D behave as expected; Escape still cancels
• On Windows/Linux: confirm Ctrl shortcuts still work; plain Ctrl must not also fire the Cmd bindings on macOS

[xhon-pelushi]

Thanks for the review — pushed a follow-up that switches the editor shortcuts to useHotKey (via a small useEditorHotKeys composable used from both EditSimple and EditFull).

• Escape / Ctrl|Cmd+Enter / Ctrl|Cmd+Delete / Ctrl|Cmd+D are registered through @nextcloud/vue/composables/useHotKey
• On macOS that means Cmd only (not Ctrl||Meta)
• The mixin window keydown listeners and handlers are removed

Happy to adjust further if needed.

[SebastianKrupinski]

The editor mixin is already a common library between both editors, why do we now need to extract and abstract this in to a composable? useHotKey is already a composable, so we are wrapping a @nextcloud composable in a local composable?

[xhon-pelushi]

Agreed — the local wrapper is gone. EditFull / EditSimple now call @nextcloud/vue useHotKey directly from setup(); useEditorHotKeys.js was removed.

[xhon-pelushi]

Thanks @odzhychko and @SebastianKrupinski — updated to call useHotKey directly from EditFull / EditSimple setup() (no local wrapper composable). macOS gets Cmd via useHotKey; Ctrl is not also accepted for those bindings.

[odzhychko]

@xhon-pelushi #8720 (comment) is still open.

Just inline the logic into setup in EditorMixin (or is there a reason why it doesn't work?). Also no need for the new code comments.

[codecov]

Codecov Report

❌ Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/mixins/EditorMixin.js	82.35%	3 Missing ⚠️

📢 Thoughts on this report? Let us know!

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

[SebastianKrupinski]

@xhon-pelushi #8720 (comment) is still open.

Just inline the logic into setup in EditorMixin (or is there a reason why it doesn't work?). Also no need for the new code comments.

I made the changes.

[SebastianKrupinski]

Note:

Shortcut keys work, but not so well for recurring events

Out dated

[SebastianKrupinski]

/backport to stable6.6

[SebastianKrupinski]

/backport to stable6.5