Cursor location in Paste from Word and Paste dialogs is incorrect for BIDI languages

[Satya Minnekanti]

To reproduce the defect:

1. Open ​http://nightly.ckeditor.com/5388/_samples/ui_languages.html

2. Select Arabic or Hebrew in the languages drop don list

3. Click on Paste from Word or Paste Icons in the Toolbar

Expected Result: Paste from Word or Paste dialog is opened and the Cursor is on the right-side of the text area

Actual Result: Paste from Word or Paste dialog is opened and the Cursor is on the left-side of the text area.

The Cursor Position is shown properly in Paste as Plain Text dialog.

[Frederico Caldeira Knabben]

Considering that the pasting area is there to receive contents to be inputted in the editor, I think the ​contentsLangDirection setting should be used instead. In that case, the "lang" attribute cannot be used as we don't have such information.

[Frederico Caldeira Knabben]

#5485 is introducing the contentLanguage setting which could be then used for the "lang" attribute.

[Garry Yao]

Matches my first thought on this, but does that means we should reflect the same thing to Paste Text dialog?

[Garry Yao]

Fixed with [5451].

[Satya Minnekanti]

This is still not fixed..Cursor is still showing on the left hand side of Text Area in Paste and Paste from Word dialogs.

[Garry Yao]

Now the paste area behaves just like editing document, explicit definition of 'contentsLangDirection' is required if you're using a ltr UI language.

[Satya Minnekanti]

There is still an inconsistency here. The textarea in the Paste Text dialog is taking the language direction from the UI, it should now use the contentsLangDirection instead.

[Garry Yao]

Align the 'Paste Text' area direction.

[Frederico Caldeira Knabben]

The documentation changes may not be needed though, as the pasting area is in fact related to the contents, so it simply inherits its direction (it may happen in other parts of the editor, even if not documented).

[Garry Yao]

Fixed with [5466].

[Damian]

This still doesn't seem to work for Paste as Plain Text. I can see that the following code is in the dialog:

'dir=' + editor.config.contentsLangDirection + ';' +

but the output HTML does not use the 'dir' property?

[Alfonso Martínez de Lizarrondo]

The patch used the "dir=" attribute instead of the css "direction:"

[Alfonso Martínez de Lizarrondo]

Fixed with [5507]