Unable to unbold text that has inline style

[Michael Camden]

I've confirmed this in Chrome on Windows 7, not sure of other browsers. If text is bolded using the strong tag, and has an inline style, it cannot be unbolded.

Steps to reproduce:

1. Goto ckeditor.com/demo and clear out the content
2. Press the 'source' button, and insert the following html:

<p>

<strong style="color:#40e0d0;">The quick brown fox.</strong></p>

3. Press the 'source' button again to enter WYSIWYG mode.
4. Select the text 'The quick brown fox' and press the 'bold' button. The text will not be unbolded.

UPDATE: Both original issue and the one from comment:5 can be reproduced

[Jakub Ś]

Issue has been reproducible in all browsers from CKEditor 3.0.

[Michael Camden]

I have a possible patch for this bug.

File: _source/plugins/styles/plugin.js Line: 1158

		// If the style attributes defined don't apply to this style element, convert to span
		if(removeEmpty && !styles && element.hasAttributes() && element.getAttribute('style')) {
			element.renameNode('span');
		}

This patch works with the following test cases;

<p>
<strong>Bolded text with no other style applied</strong>
</p>
<p>
<strong style="color: red">Bolded text with a red color style</strong>
</p>
<p>
<strong><span style="color: red">Bolded text with red color style in span</span></strong>
</p>

This patch should also resolve similar bugs with other style elements like 'em'. In fact, in my testing the following similar test cases are also work:

<p>
<em>Italic text with no other style applied</em>
</p>
<p>
<em style="color: red">Italic text with a red color style</em>
</p>
<p>
<em><span style="color: red">Italic text with red color style in span</span></em>
</p>

[Jakub Ś]

#6026 was marked as duplicate.

I believe this issue is related to #8686 and perhaps a little bit to #9295.

[Jakub Ś]

This is no longer an issue in CKEditor 4.x. I have tested this in latest CKE 4.1 and unbolding worked as expected.

[Kevin Berridge]

This bug is NOT fixed in the latest CKE 4.1.1 (at the time of this writing). You can reproduce it on the demo page as follows:

1. Clear all the text
2. Change the format to Normal
3. Type in 3 headings with an empty paragraph below them like:

H1

H2

H3

4. Bold each heading
5. Put your cursor in front of H2, shift + arrow up, arrow, up to select H1.
6. Delete the selected test
7. Put your cursor at the end of H2, hit enter
8. Type something, notice it is bold.
9. Try to unbold the text you just typed

RESULT: It refuses to unbold.

NOTE: If you click the "Source" button, then switch back to WYSIWYG, it will start working. This appears to be why the original steps to reproduce in the op's post work now. But in fact, the bug is not completely fixed.

[Jakub Ś]

TC mentioned in comment:5 can be reproduced from CKEditor 3.5.3 rev. [6461] (when applying inline styles for collapsed selection in Webkit was implemented) and is reproducible in both branches 3.x and 4.x.

This TC is only reproducible in Chrome - other browsers don’t seem to add any inline styles.

[Kevin Berridge]

Any update on this issue? It's a pretty severe problem for our clients.

Thanks.