[FF] Exception after inserting whitespace while inline style is active

[Mgsy]

🐞 Is this a bug report or feature request? (choose one)

• Bug report

💻 Version of CKEditor

1.0.0-alpha2

📋 Steps to reproduce

1. Go to http://localhost:8125/ckeditor5-core/tests/manual/article.html.
2. Put the caret at the end of the Heading 1.
3. Press Spacebar.
4. Press Arrow left.
5. Press Ctrl + B.
6. Press Spacebar.

✅ Expected result

No error in the console.

❎ Actual result

There is the error in the console.

📃 Other details that might be useful

Error

TypeError: viewBlock is null mapper.js:86
   Mapper/< mapper.js:86
   fire emittermixin.js:269
   toModelPosition mapper.js:195
   _handleTextNodeInsertion input.js:298
   handle input.js:152
   _handleMutations input.js:105
   init/< input.js:50
   fire emittermixin.js:269
   _onMutations mutationobserver.js:248
   _onMutations self-hosted:954

GIF

Notes
This bug occurs also if you put the selection at the beginning of the element. It's not reproducible when the selection is inside the caption.

Other information
OS: Windows 10, MacOS X

[ma2ciek]

Something probably goes wrong with the Input plugin. The mutations change after InputCommand and the node's parent disappears. Also, it should be noted, that MutationObserver on the FF emits 2 view mutations, while on the Chrome emits only 1.

[Reinmar]

We had a talk about this issue back in December and we identified two problems there:

1. That DomConverter._processDataFromDomText() loses the first space after <br> (or something similar – basically, it doesn't consider the <br> so it thinks that that space is a trailing space and trims it).
2. That mutation observer doesn't correctly filter out <br> mutations in these cases.

Do I recall this correctly, @ma2ciek @szymonkups? PS. Always write down such notes right after a talk.

[ma2ciek]

I remember, that we've been talking about the second, similar problem (#692) and our talks are concluded here: #692 (comment).

[ma2ciek]

Notes for the 1. scenario:
Step 1. applies to paragraphs too.
At step 6, it can be any character.

2. scenario for that error:
3. Go to http://localhost:8125/ckeditor5-core/tests/manual/article.html.
4. Put the caret at: Para[]graph.
5. Press Spacebar twice.
6. Press Arrow left.
7. Press Ctrl + B.
8. Press any character.

It's different than #692. Here, all is about the bogus <br> tag IMO.

[CKEditorBot]

There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may be relevant, so if you're interested in the solution, leave a comment or reaction under this issue.

[CKEditorBot]

There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may still be relevant, so if you're interested in the solution, leave a comment or reaction under this issue.

[CKEditorBot]

We've closed your issue due to inactivity over the last year. We understand that the issue may still be relevant. If so, feel free to open a new one (and link this issue to it).