elastic / eui

Elastic UI Framework 🙌
https://eui.elastic.co/
Other
6.07k stars 821 forks source link

[EUICopy] [copyToClipboard] Consider updating `document.execCommand('copy')` to use the `Clipboard API` #6912

Closed 1Copenut closed 2 weeks ago

1Copenut commented 1 year ago

Describe the solution you'd like Our copyToClipboard() service use the deprecated document execCommand API for copying snippets to a user's clipboard. @P1llus suggested the Clipboard API as an alternative.

The API appears well-supported on first review, but may have some issues in Firefox. This is worth a discussion by the team.

The service is used primarily in EuiCopy, based on our Slack discussion.

Is your feature request related to a problem? Please describe. document.execCommand('copy') is deprecated and may be dropped from browsers in the future.

github-actions[bot] commented 1 year ago

👋 Thank you for your suggestion or request! While the EUI team agrees that it's valid, it's unlikely that we will prioritize this issue on our roadmap. We'll leave the issue open if you or anyone else in the community wants to implement it by contributing to EUI. If not, this issue will auto close in one year.

Log | Bot Usage
github-actions[bot] commented 6 months ago

👋 Hi there - this issue hasn't had any activity in 6 months. If the EUI team has not explicitly expressed that this is something on our roadmap, it's unlikely that we'll pick this issue up. We would sincerely appreciate a PR/community contribution if this is something that matters to you! If not, and there is no further activity on this issue for another 6 months (i.e. it's stale for over a year), the issue will be auto-closed.

github-actions[bot] commented 2 weeks ago

❌ Per our previous message, this issue is auto-closing after having been open and inactive for a year. If you strongly feel this is still a high-priority issue, or are interested in contributing, please leave a comment or open a new issue linking to this one for context.