Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The attached patch should fix two issues which seem to have been introduced recently in the JS code in fckeditor.popup.html.
1. A leftover debug line was preventing content from being transferred from the editor to the Body field when "OK" was clicked in FF and IE at least.
2. Editing block content wasn't working because the editor was trying to put text in a non-existent teaser field.
Comment | File | Size | Author |
---|---|---|---|
fckeditor-bugs.patch | 1.69 KB | Garrett Albright |
Comments
Comment #1
Jorrit CreditAttribution: Jorrit commentedYou're absolutely right about the console.log. I didn't check for teaser === null, because I only attach the teaser GET parameter when there is a teaser. But checking again doesn't hurt, I think. Is the
===
comparison also valid in JS? I thought it was only valid in PHP.Comment #2
Garrett Albright CreditAttribution: Garrett Albright commentedSure it is. Try running
1 == '1'
and1 === '1'
in the Firebug console or similar and you'll see it behaves as it does in PHP.Comment #3
Jorrit CreditAttribution: Jorrit commentedI have committed a fix, slightly different than your patch. You will be able to find it in the next -dev release, which are built nightly. I hope you like it.
Comment #4
Jorrit CreditAttribution: Jorrit commented