[Bug]: copy / paste (all) buggy (macOS)

nfdi4plants / Swate

Excel Add-In for annotation of experimental data and computational workflows.

https://swate-alpha.nfdi4plants.org

MIT License

30 stars 6 forks source link

[Bug]: copy / paste (all) buggy (macOS) #389

Closed Brilator closed 4 months ago

Brilator commented 4 months ago

Contact Details

issue

What happened?

There's mixed behavior with the copy paste functionality between cells.

In Firefox, I always get the error navigator.clipboard.readText is not a function upon paste and paste all; independent of how many cells I copy
In Safari, after copying multiple cells, both paste and paste all only past one cell
paste / paste all work fine in MS Edge and inside ARCitect

What did you expect?

No response

Host

Standalone

Host Details

macOS

Browser Safari
Browser Firefox

Swate version

v1.0.0-alpha.15

Freymaurer commented 4 months ago

Ahh yes, firefox only supports it in a alpha version

For Safari could you pls paste the your copied cells into excel or any .txt file? This way we can find out if there is something fishy about the copy or the paste

Brilator commented 4 months ago

Ugh, apparently I cannot. Can't reproduce the error in Safari. All fine now.

Brilator commented 4 months ago

Copy / paste to txt or excel also works fine.

Freymaurer commented 4 months ago

To be honest, i am not sure how much time i should spend on adding a full alternative for firefox 😞

The browser api i chose seems to be the current best practise and is so much cleaner than alternatives like:

create invisible input element with a value of what needs to be copied
programmatically select the value and copy it
remove the invisible element

The feature was added to firefox nightly in version 122 (experimental features), released 2023-11-21. So i hope they will soon enable it by default for standard release.

I will close this issue for now. If this creates friction with users, please let me know, then i will implement an alternative solution 🙂