Test navigator.clipboard defined before accessing

[Xadiant]

Most browsers now require that the the web page is "Secure" or else they do not allow access to the clipboard, this includes navigator.clipboard. I noticed that in my application that when I access my app locally (where https is not applied yet) invoking the copy methods results in an exception where "writeText" does not exist in "undefined".

To smooth over this, I added a test for the clipboard existing before calling the method.

If you agree with this idea, I imagine we will want to output an error when this if check fails, or maybe modify the function signature to return a variable indicating success.

[slowriot]

Thanks for the contribution. I think it needs a bit more, as currently it's just suppressing the error silently. Ideally, as you say the function should have some way of reporting the error - returning bool instead of void so developers can identify if the operation failed, along with a console log message explaining why for the user's benefit.