Does not work for me without this patch.

Do not forget to put this filter "Cleanup NBSP markup" last in "admin/config/content/formats/manage/full_html"

CommentFileSizeAuthor
#6 3081167-06.patch683 byteswengerk
nbsp-filter.patch688 bytesmgerbault

Comments

mgerbault created an issue. See original summary.

mgerbault’s picture

Issue summary: View changes
phily’s picture

Thanks, I had to apply above patch too (Drupal 8.7.8, NBSP 8.x-1.2, basic_html text filter)

wengerk’s picture

Status: Active » Needs work
Issue tags: +Need tests

Hey !! Many thanks for your help, could you please provide a patch to expose the main issue and be sure we never create a regression on this ?

wengerk’s picture

The module has now tests, I don't understand how it does not works before ? LEt's see what's broken with the change you made and how we can improve the module !

wengerk’s picture

StatusFileSize
new683 bytes

Reroll the patch

wengerk’s picture

Status: Needs work » Postponed (maintainer needs more info)

The tests still pass, but they also pass without your fix :/ I'm kind of confuse could you please provide more context how to repeat the problem ?

It was a problem with the tests suites, may bad.

So the new code don't pass we have to see why and be sure we really fix something with the patch proposed and cover the case.

renrhaf’s picture

We had the same issue, the HTML generated was like this :
.....<span class="nbsp"> </span>....
So when the filter was replacing by inner child, there was nothing but a simple space character.

wengerk’s picture

Status: Postponed (maintainer needs more info) » Fixed

fixed on new release 8.x-2.2 (CKEditor 4 & CKEditor 5) and 3.0.0 (CKEditor 5).

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.