Indenting commands in table cells creates collapsed paragraphs.

[martinkou]

To reproduce:

1. Open replacebyclass.html in Firefox/Safari/Opera.
2. Create a table.
3. Move caret to first table cell.
4. Indent.
5. Move caret to second table cell.
6. Try to move caret back to first table cell.
7. You can't do that (Firefox), or you'll have strange selection positions in other browsers.

[garry.yao]

We've again tripped by bookmark ignoring logic, and this time is about CKEDITOR.dom.element::getNext/getPrevious, I'm proposing of introducing a guard function to it as we deal with 'getPreviousSourceNode/getNextSourceNode'.

[fredck]

There are a few issues in the getNext and getPrevious changes:

• The "guard" name for the function is misleading, as it doesn't have the same usage we have in the walker to "stop" the search. It instead acts like a filter, and we have used the "evaluator" name for it in the walker. The same name should be used here.

• The CKEDITOR.dom.element class is being used to transform the node when passing it to the guard. We could have text nodes also there, so the CKEDITOR.dom.node class should be used.

• The raw DOM node is being transformed twice, once when passing it to the guard (evaluator) and again when returning the function result. We should find a solution to make this transformation only once (maybe saving the created object into a new variable).

Other than that, we must be sure that the changes in the function are not impacting other parts of the code which are already using it.

[garry.yao]

Beside fixing the above problems, now evaluator is symmetrically available on 'CKEDITOR.dom.element.getFist/Last', adding TCs for changed function.

[fredck]

Let's simplify the implementation just a little bit. Something like this:

getNext : function( evaluator )
{
	var next = this.$, retval;
	do
	{
		next = next.nextSibling;
		retval = next && new CKEDITOR.dom.node( next );
	}
	while ( retval && evaluator && !evaluator( retval ) )
	return retval;
}

Also, I understand the theoretical benefit for the getFirst and getLast changes, but we're not using this new feature anywhere, so we should not have it at this time. We may introduce it in the future at first use. Please revert those changes.

[garry.yao]

Fixed with [3930].