Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
With the Comium browser it is not possible to insert images.
This error occurs when inserting into comments and terms.
Reason:
// Add the click handler to the insert button.
$('.insert-button', context).unbind('click').click(insert);
This code works with Cromium:
// Add the click handler to the insert button.
$('.insert-button', context).click(insert);
Postscript
Curently it is not possible to insert images of comments and terms! Way out: Fix the limit of insertion of images only with nodes.
Comment | File | Size | Author |
---|---|---|---|
#7 | insert_double_fix.patch | 766 bytes | quicksketch |
#2 | cromium-insert.js_.patch | 508 bytes | quiptime |
Comments
Comment #1
quiptime CreditAttribution: quiptime commentedSorry for the double post.
Comment #2
quiptime CreditAttribution: quiptime commentedWith the patch it is possible to work with the Cromium browser to insert images on comments and terms.
Note
To test the patch are required the use of Patch - allow insert action on all textareas
Comment #3
quiptime CreditAttribution: quiptime commentedbump
Comment #4
quicksketchThis patch looks fine to me. I'm not sure why we would be unbinding the click handlers first, as jQuery already prevents double binding of the same function. However, please stop bumping issues, I review my modules one at a time, it may take a few weeks for me to get to the Insert queue.
Comment #5
quicksketchI can't reproduce this problem any more in newer versions of Chromium, but regardless it's not a bad idea. Committed to both branches.
Comment #6
quicksketchThis issue caused Insert to start inserting images twice on every click. Because the AJAX handler on fields attaches once when the field is added, and then again on the entire form, the click handlers end up getting bound twice.
Comment #7
quicksketchI've committed this patch which takes the classic approach of setting guard classes to prevent double-binding.