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

Hide the Copy buttons in docs when Flash is not available #16301

Closed

Conversation

tarebyte
Copy link
Contributor

From issue #16269.

Feedback welcome!

@tarebyte tarebyte force-pushed the remove_copy_button_on_flash_error branch from edf4a9c to 88d302f Compare April 16, 2015 00:29
@cvrebert
Copy link
Collaborator

Could you remove the ZeroClipboard.min.map file?

@cvrebert
Copy link
Collaborator

Please fix the JS coding style errors that are making the Travis build fail (see the Travis log for details).

@cvrebert cvrebert added this to the v3.3.5 milestone Apr 16, 2015
@cvrebert
Copy link
Collaborator

So, when Flash is unavailable, the first time the user presses a Copy button, all the Copy buttons will disappear?

@tarebyte
Copy link
Contributor Author

@cvrebert the user never sees the copy button if flash is not supported

@cvrebert
Copy link
Collaborator

Great, was just making sure since I'm not too familiar with ZeroClipboard.

@tarebyte
Copy link
Contributor Author

@cvrebert is there a way to optimize the JS so that you can't see the button appear and then dissapear?

This is what I run into sometimes.

copy-button

@cvrebert
Copy link
Collaborator

@tarebyte Not that I know of.

@cvrebert
Copy link
Collaborator

Just tested this out; LGTM.

@cvrebert cvrebert closed this in 229e888 Apr 17, 2015
@cvrebert cvrebert mentioned this pull request Apr 17, 2015
@tarebyte
Copy link
Contributor Author

💖

@tarebyte tarebyte deleted the remove_copy_button_on_flash_error branch April 17, 2015 00:25
@cvrebert
Copy link
Collaborator

Thanks for this improvement! <3

@XhmikosR
Copy link
Member

@tarebyte @cvrebert: there was a reason we sticked to that older version. And that is that the newer versions do not work with IE < 9.

@XhmikosR
Copy link
Member

Personally, I'd appreciate a CC before this kind of stuff.

@cvrebert
Copy link
Collaborator

Sorry about the CC; didn't know you were interested.

And that is that the newer versions do not work with IE < 9.

In retrospect, we should have documented that better...

@XhmikosR
Copy link
Member

Well... I don't see where one could document this. IE 8 support was there,
and I purposely skipped updating this. You should find more info maybe in
the PR that introduce ZeroClipboard.

Will you do the revert or do you want me to make a PR?
On Apr 28, 2015 1:08 PM, "Chris Rebert" [email protected] wrote:

Sorry about the CC; didn't know you were interested.

And that is that the newer versions do not work with IE < 9.

In retrospect, we should have documented that better...


Reply to this email directly or view it on GitHub
#16301 (comment).

@cvrebert
Copy link
Collaborator

PR would be faster, since this is low on my priority list.

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

Successfully merging this pull request may close these issues.

3 participants