Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy to clipboard button #2698

Closed
yurikuzn opened this issue Apr 18, 2023 · 10 comments
Closed

Copy to clipboard button #2698

yurikuzn opened this issue Apr 18, 2023 · 10 comments
Assignees
Milestone

Comments

@yurikuzn
Copy link
Contributor

A parameter for the following field types:

  • Varchar
  • URL
  • Number

image

@yurikuzn yurikuzn added this to the Version 7.5.0 milestone Apr 18, 2023
@yurikuzn yurikuzn self-assigned this Apr 18, 2023
@yurikuzn
Copy link
Contributor Author

3d743d3

@Kharg
Copy link
Contributor

Kharg commented Apr 18, 2023

Awesome!
Would it be difficult to extend it to email/phone/text fields?

@yurikuzn
Copy link
Contributor Author

There are some difficulties.

@o-data
Copy link
Contributor

o-data commented Jun 14, 2023

What about the new field: Multi-URL?

@yurikuzn
Copy link
Contributor Author

yurikuzn commented Jun 14, 2023

Not planned for now.

@RabeeBrahimi
Copy link

i think it will better if we check if browser support clipboard if not then hide the icon, no need to show the copy to clipboard icon when the current browser doesn't support such functionality.

@yurikuzn
Copy link
Contributor Author

yurikuzn commented Jun 14, 2023

What browsers do not support navigator.clipboard ? According this https://caniuse.com/mdn-api_navigator_clipboard, all do support.

@RabeeBrahimi
Copy link

RabeeBrahimi commented Jun 14, 2023

so i have tried it on firefox version113 which the latest and i get an error. see attached screenshot. also tested it on chrome version 114.0.5735.110 same error see screenshot
Not sure if the error is only local or is this general.
i am using espocrm version 7.5.1

chrome_clipboard_error clipboard_error

@yurikuzn
Copy link
Contributor Author

It requires either localhost or HTTPS.

@RabeeBrahimi
Copy link

yes you are right, switched to https and it works. i am not using localhost (using laravel valet instead)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants