Skip to content

Conversation

@Kully
Copy link
Contributor

@Kully Kully commented Aug 10, 2018

For Issue #856

@jdamiba
Copy link
Contributor

jdamiba commented Aug 24, 2018

I was able to successfully build this branch on my local machine and saw that the "Copy to clipboard!" is indeed in visible. However, on my machine the box is just text, and not clickable to add the text to my clipboard. Is that related to how the assets are loaded in a localhost environment vs production?

@Kully
Copy link
Contributor Author

Kully commented Dec 4, 2018

Report of the Issue so far:

Google (Chrome) has been moving away from flash over the years to the point that you need to allow flash to play on most websites.

The copyCode buttons rely on flash so this means that we need to allow each time, unless the button has a bigger resolution that 400X300 (not gonna happen).

If I do allow flash to run on the page, I am still hitting an error that I am working to fix now, but I am wondering if we should be moving away from flash altogether? I can't imagine anyone who would want to enable flash on Chrome before simply clicking a simple Copy button.

Thoughts?

@Kully
Copy link
Contributor Author

Kully commented Dec 5, 2018

@Kully
Copy link
Contributor Author

Kully commented Dec 18, 2018

clipboard-works

@Kully
Copy link
Contributor Author

Kully commented Dec 18, 2018

@cldougl ready for a review - clipboard works now.

NOTE: I've only tested in Chrome. I'll test in Safari as well to be very sure.

@Kully Kully requested a review from cldougl December 18, 2018 14:19
@cldougl
Copy link
Member

cldougl commented Dec 18, 2018

@Kully can you use browserstack and test all browsers?

@Kully
Copy link
Contributor Author

Kully commented Dec 18, 2018

Testing browsers:

  • Firefox
  • Opera
  • IE11
  • Edge
  • Chrome
  • Safari
  • Yandex

Only isn't working in Microsoft Edge

It looks like it is a sandboxing issue

screen shot 2018-12-18 at 4 07 14 pm

@nicolaskruchten
Copy link
Contributor

@michaelbabyn can you see if this works/can be merged please? copy-to-clipboard right now in prod does NOT work in Chrome/OSX for me

@michaelbabyn
Copy link
Contributor

This works for me in chrome and firefox although on Chrome it's not clear without opening dev tools that you need flash set up and enabled for the button to work.

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

Labels

None yet

7 participants