-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Dragging image into wiki pages is broken #5650
Comments
Yes! It's the same as this: #5229 (comment) |
I'm currently unable to click to upload or drag an image onto a wiki page. I'm not sure if it's related. I'm not getting an error, just no response no matter how long I let the webpage sit open and "think" about it. |
Same issue here on /air-quality |
I also tried this post clearing-cache, in a different browser, and on a different computer. Still no image uploading working for me.
… On May 10, 2019, at 10:02 AM, Stevie ***@***.***> wrote:
Same issue here on /air-quality
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#5650 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ALUKWT6MJCUVOPHAPUAEDH3PUWE77ANCNFSM4HKB2NQA>.
|
This was fixed in #5657 for comment image uploading, but the fix didn't apply to wiki page image upload. Checking why now; I believe it is failing on // wraps currently selected text in textarea with strings a and b
wrap: function(a,b,args) {
this.refresh();
var len = $E.textarea.val().length; That's here: plots2/app/assets/javascripts/editor.js Line 43 in a3e40c5
It's initialized here: plots2/app/views/editor/_editor.html.erb Line 39 in a3e40c5
|
Yes!!! This changed, re-assigning https://github.com/publiclab/plots2/pull/5321/files#diff-fd61d47ff16a49707d50fa5f9ae7c5f5R32 We need to make an exemption for wiki image upload. |
Filed a fix, which I tested, at #5703! |
Thanks, all! |
I'm having trouble dragging an image onto a wiki edit page; it returns an error on this line of dragdrop.js:
($D.selected).find('#create_progress').eq(0).show();
Uncaught TypeError: Cannot read property 'find' of undefined
I wonder if
$D.selected
was affected somehow? This is the kind of thing a system test (#5316) will help prevent!Noting this could be linked to #5229
The text was updated successfully, but these errors were encountered: