Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
See #736066: ajax.js insert command sometimes wraps content in a div, potentially producing invalid HTML and other bugs, for the full write up. But basically ajax insert will wrap new content in a div, which is bad (it was a workaround for Safari 3 it sounds like).
anyway, i stole the fix from comment #106 of #736066: ajax.js insert command sometimes wraps content in a div, potentially producing invalid HTML and other bugs.
patch to come...
Comment | File | Size | Author |
---|---|---|---|
#4 | ajax_js_insert_command-2771521-4.patch | 673 bytes | thejacer87 |
Comments
Comment #2
thejacer87 CreditAttribution: thejacer87 at Acro Commerce commentedhere's the patch
Comment #4
thejacer87 CreditAttribution: thejacer87 at Acro Commerce commenteddunno why it failed... maybe cuz i made the patch in my own project so the commits were off?
Comment #5
thejacer87 CreditAttribution: thejacer87 at Acro Commerce commentedComment #6
stefan.r CreditAttribution: stefan.r commentedThis probably needs to be fixed in 8.x first (see core/misc/ajax.js).
Comment #7
stefan.r CreditAttribution: stefan.r commentedSeems that we already have an 8.x issue at #736066: ajax.js insert command sometimes wraps content in a div, potentially producing invalid HTML and other bugs.
In order for these kinds of fixes to go into 7.x, they need to go into 8.x first.
Comment #8
thejacer87 CreditAttribution: thejacer87 at Acro Commerce commentednot sure i understand. i mentioned that exact issue in my original post... twice. why do we have to wait for 8?
Comment #9
poker10 CreditAttribution: poker10 at ActivIT s.r.o. commentedThe D8 issue is commited, so that fix can be backported here to D7. Patch will need some work to include all (possible) changes from D8 commit.