Continuing #876694-36: Teaser disappears when using CKEditor as requested.
Works:
This is a paragraph.
Doesn't work:
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Curabitur massa urna, tristique quis suscipit id, porttitor id nunc. Praesent quis neque ante. Praesent aliquet ullamcorper ipsum, ac fermentum orci dapibus ut. Nullam vel sapien in enim congue luctus et nec sem. Proin id augue metus. Maecenas faucibus tempor erat et molestie. Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nunc quis nunc turpis. Donec pulvinar commodo tempor. Aliquam erat volutpat. Duis purus urna, porta quis porta non, ullamcorper commodo eros. Proin molestie tristique mattis. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos himenaeos. Praesent et diam nec orci adipiscing fermentum non viverra mi. Quisque porta mollis tempor. Maecenas non ipsum justo. Nullam nulla lorem, cursus sit amet fermentum ut, congue quis velit.
Reason:
CKEditor reformats the HTML to contain a newline directly after the opening <p>. When the new line filter is enabled,node_teaser() will use line breaks as part of its calculations and fail to properly break the first paragraph leaving only an empty p tag.
If you can, the workaround is to disable the newline filter for any input format used with ckeditor.
Comment | File | Size | Author |
---|---|---|---|
#3 | ckeditor_p_breakafter_linebreakfilter_fix.patch | 718 bytes | neclimdul |
#2 | ckeditor_p_breakafter_linebreakfilter_fix.patch | 800 bytes | neclimdul |
Comments
Comment #1
neclimdulApparently I can't use that workaround so I'm now investigating a real solution...
Comment #2
neclimdulRelated ckeditor list discussion:
http://dev.ckeditor.com/ticket/3260
Configuration fix for us:
Comment #3
neclimdulerr... not a lazy patch created of my production code...
Comment #4
dczepierga CreditAttribution: dczepierga commented@neclimdul, i add this lines in other place and commit it to GIT so pls download the last DEV version and check does it works for u.
Greetings
Comment #5
dczepierga CreditAttribution: dczepierga commentedI also do this in module for Drupal 7: #1183218: [D7] Teasers for long paragraphs broken
@neclimdul, really thx for good work and fix it.
Greetings
Comment #6
neclimdulYou could have just used the patch to be ported status and I'd been happy to help. Thanks though!
Comment #7
mkesicki CreditAttribution: mkesicki commented