[Safari] `copy_with_column_headers` context menu option doesn't work

[alojzy231]

Describe the bug

Hi!
I've got an error on Safari v18.1 where I want to use context menu's copy_with_column_headers option. It doesn't copy any value, column header nor cell's value (doesn't add any value to the clipboard). When using cmd + c it works.
I've tested an example in Handsontable's docs: All available Handsontable hooks example and it seems that using copy options from context menu doesn't trigger beforeCopy and afterCopy hook but they get triggered on cmd + c.

I've seen that there has been implemented a workaround in CopyPaste plugin in #ensureClipboardEventsGetTriggered method for Safari and was tested on Safari@16.5.2.

I believe that is a bug.

Is there any workaround that I can implement for to have an option to copy cells' with their columns' headers while waiting for the fix?

Video/Screenshots

No response

Provide a link to the demo with the bug reproduction

https://stackblitz.com/edit/react-bjedvh?file=src%2FExampleComponent.jsx

Handsontable version

14.6

Framework version

No response

Your environment

Safari@18.1

[adrianszymanski89]

Hi @alojzy231

Our documentation contained misinformation regarding defining this option (and others related to copy column headers), and we added the needed information recently.

In short, to make this option work correctly from the contextMenu level, you have to configure the copyPaste plugin in the options. Here's the correct configuration: https://stackblitz.com/edit/react-bjedvh-jmjmd2?file=src%2FExampleComponent.jsx

This section now contains the correct information about the additional steps needed: https://handsontable.com/docs/javascript-data-grid/context-menu/#context-menu-with-selected-options

[alojzy231]

Hi @adrianszymanski89, thanks for the reply!

I've checked your example on Safari@18.1 and it still doesn't work. Could you check it? It works properly on chromium browsers

[adrianszymanski89]

Hi @alojzy231

It appears it executes the wrong option on Safari - copy_column_headers_only. Thank you for catching this. I will report it internally and let you know once it's fixed.