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

Clipboard disabled when Flash isn't available #1876

Merged
merged 7 commits into from
Nov 11, 2014
Merged

Conversation

w33ble
Copy link
Contributor

@w33ble w33ble commented Nov 10, 2014

Closes #1768

Don't show the clipboard if ZeroClipboard.isFlashUnusable() is true

  • Change how ng-clip and ZeroClipboard are included
    • Include note about why ZeroClipboard is bound to window
  • Add disabled to scope, use it to hide the directive
  • Adjust for coding style

@w33ble w33ble added the review label Nov 10, 2014
@spalger
Copy link
Contributor

spalger commented Nov 11, 2014

LGTM

spalger pushed a commit that referenced this pull request Nov 11, 2014
Clipboard disabled when Flash isn't available
@spalger spalger merged commit 67f3fe9 into elastic:master Nov 11, 2014
@spalger
Copy link
Contributor

spalger commented Nov 11, 2014

@rashidkpc Sorry, got trigger happy. Hope it's okay that this goes into Beta 2

spalger pushed a commit that referenced this pull request Nov 11, 2014
This reverts commit 67f3fe9, reversing
changes made to 1b3a385.
@rashidkpc
Copy link
Contributor

-1 reverting this, seems to break entirely?

spalger pushed a commit to spalger/kibana that referenced this pull request Nov 12, 2014
@spalger spalger mentioned this pull request Nov 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable copy to clipboard without flash
3 participants