config:startupFocus not working

[Garry Yao]

CKEDITOR.config.startupFocus set to 'true' doesn't work for both modes.

[Garry Yao]

Functional test case

[Artur Formella]

Ok. I have 2 solutions: light and heavy. Light version sometimes works, sometimes not, sufficient in most cases. Heavy version can be used with scripts and checked with test case. I also attached startupfocus_test. It works only with "heavy" version.

[Frederico Caldeira Knabben]

• The "heavy" solution is concentrated on passing the test case, not really fixing this bug. The problem is that the test case has been made in a way that it depends on several other features in the editor, instead of being limited to test the issues related to this ticket. It even do that in was not supported by the editor currently. So both the test case and the patch are inappropriate.

• Even though, the "heavy" patch proposes things that are interesting for us to have, like the "onLoad" feature. As I've pointed you in an e-mail, I would not use onLoad for that, because it still makes it not possible to register to events that happen even before the total editor load (like "pluginsLoaded", for example). I've opened #3173 for such feature.

• I've noted that the "heavy" patch tries to fix some focus issues. If such issues have been found, please open a dedicated ticket for that. But, I would not use the "instanceReady" for any focus check, using instead the "focus" event.

The "light" solution is ok, but recent changes to the code invalidate it. I'm taking over the ticket to propose a new fix.

[Frederico Caldeira Knabben]

Fixed with [3266].