The changes are not limited to the text selection when using the inline CKEDITOR

[jckaashoek]

When using the inline CKEDITOR and texttransform, the updates are not limited to the selected text, the entire paragraph appears to get changed.

[saugataroy]

I've also faced this bug. In inline mode the buttons do not respond properly and the entire paragraph gets transformed.

[jckaashoek]

Fixed the issue, here is an example one of the code snippets

editor.addCommand('transformTextToUppercase',
{
exec : function()
{
var selection = editor.getSelection();
if (selection.getSelectedText().length > 0) {
var theText = selection.getSelectedText();
editor.insertText(theText.toLocaleUpperCase());
}
} //func
});

[saugataroy]

Thanks! it is working perfectly now.

[dlorin]

But if the selection contains some strong or background text, the style becomes "normal". You have to go thru the nodes using walker.next().

You just have to stop the transformation at the end of the selection (with "longueur")

var transformSelectedText = function(editor, transformFunc) {

    var selection = editor.getSelection();
    longueur = selection.getSelectedText().length;
    if (longueur > 0) {
	selection.lock();
	var range = selection.getRanges()[0], walker = new CKEDITOR.dom.walker(range), node, nodeText;

	while ((node = walker.next())) {

	    if (node.type == CKEDITOR.NODE_TEXT && node.getText()) {
		nodeText = node.$.textContent;
		if (node.equals(range.startContainer)) {
		    nodeText = nodeText.substr(0, range.startOffset) + transformFunc(nodeText.substr(range.startOffset, longueur))
			   + nodeText.substr(range.startOffset + longueur);

		} else if (node.equals(range.endContainer)) {
		    nodeText = transformFunc(nodeText.substr(0, range.endOffset)) + nodeText.substr(range.endOffset);
		} else {
		    nodeText = transformFunc(nodeText);
		}
		node.$.textContent = nodeText;
	    }
	}
	selection.unlock(true);
    }
}

[Lokavidu]

@dlorin thank you, your modification to the function solves the issue! the plugin is now working perfectly for inline editor again. thanks

[onderceylan]

Hey @dlorin, why don't you create a PR with this change? It looks good and validated already.