-
Notifications
You must be signed in to change notification settings - Fork 26
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
centerInViewport is not a function #3
Comments
Just ran into this and had to use HEAD^ for it to work. |
I have the same problem. I believe I'm using HEAD but it still doesn't work. Any ideas? |
I just installed from HEAD and same problem. Commenting out the offending line works but the result doesn't look great (It is usable though) |
Of course, I am on 0.8 so just changing that line back to the old way works nicely. Maybe a 0.8 and 0.9 branch? |
After you clone the repo checkout commit 94a4f09 and you'll be all set. |
in http://github.com/pilu/radiant-copy-move/commit/00ad9584c0fef5edeb89932c0b83d1a3ff85510a
the js was updated to use centerInViewport to center the dialog. This function does not exist in standard prototype or jQuery and thus breaks this plugin for anyone not using the custom library. This library this function is from needs to be explicitly added as a dependency or this change should be reverted.
The text was updated successfully, but these errors were encountered: