Support window name in the popup plugin

[Sergiy Shyrkov]

With the following ​changes the popup plugin was extended with window features, but it is still impossible to specify the popup windowName. It would be nice to add support for it, otherwise the file browser is opened in a new window each time (even if the corresponding window is already opened).

The popup function in the CKEditor/trunk/_source/plugins/popup/plugin.js can be adjsuted to accept windowName:

popup : function( url, width, height, options, windowName )

Thank you in advance!

[Sergiy Shyrkov]

Attached the patch (ckeditor-7616.patch) against current trunk (last revision was 6829)

[Jakub Ś]

I have tried your code in latest CKEditor 4.2 with CKFinder 2.4. The result wasn't as expected. On second Browse Server button click the popup was closed. Something like this doesn't happen with native code (I have checked to make sure).

Never the less you are right users should have the choice whether they want to open new window on every button click or want just one window opened. Providing name for popup window as config option should allow that and will override default "_blank" setting.

[Sergiy Shyrkov]

Thank you for the feedback Jakub! The patch was submitted by me more than 2 years ago against trunk, which at that time was 3.x. I doubt the original patch is compatible with the CKEditor 4.x.

Replying to j.swiderski:

I have tried your code in latest CKEditor 4.2 with CKFinder 2.4. The result wasn't as expected. On second Browse Server button click the popup was closed. Something like this doesn't happen with native code (I have checked to make sure).

Never the less you are right users should have the choice whether they want to open new window on every button click or want just one window opened. Providing name for popup window as config option should allow that and will override default "_blank" setting.