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

Plugin not intialize with dynamic data #3

Closed
kmv opened this issue Jan 11, 2014 · 6 comments
Closed

Plugin not intialize with dynamic data #3

kmv opened this issue Jan 11, 2014 · 6 comments

Comments

@kmv
Copy link

kmv commented Jan 11, 2014

Condition o.is(':visible') in line 35 not works with ajax html? Is this condition required?
Thanks for plugin!

@valeriansaliou
Copy link
Owner

Yes it is, cause the code used (ZeroClipboard) needs to retrieve element to be visible in order to position the Flash element (the click needs to be done right on the embedded Flash applet, so that we can fire a Flash action to store in the clipboard the data being passed - this is the only we we can do it).

So yeah, the element needs to be visible.

@kmv
Copy link
Author

kmv commented Jan 11, 2014

But Plugin works fine (ff 26) if i removes subject code. Flash element is positioned correctly over element (absolute div with fixed sizes retrived by ajax and element are invisible at initialization state).
Maybe possible to provide a choice - to check whether or not the element is visible.

@valeriansaliou
Copy link
Owner

Let me know, are you using display: none or visibility: hidden on that element? I can provide a fix and only ignore if display: none (cause I cannot retrieve position for this one), but visibility: hidden still makes the element being placed (I can get the position even if hidden this way).

@kmv
Copy link
Author

kmv commented Jan 12, 2014

Yes i use display: none;

@valeriansaliou
Copy link
Owner

Okay so then even with changes I might not be able to get element position. Can you apply the clipboard rather right after showing the target element in your code? Might be the most straightforward way to fix it.

@valeriansaliou
Copy link
Owner

Closed as no fresh details coming in.

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

No branches or pull requests

2 participants