extra space exists in mirrored Editor

[ahmedasaleh]

Using the test_Editor.html file to test the BIDI support for the Editor, it found that there are Extra spaces are added in mirrored dijit.Editor

[ahmedasaleh]

Extra spaces are added in mirrored dijit.Editor

[ahmedasaleh]

This problem appears in IE browser, FF and Chrome are OK.

[Douglas Hays]

patch from Hossam Katory (IBM, CCLA)

[bill]

About the change to the general editor CSS:

.claro .dijitEditorRtl .dijitEditorIFrameContainer {
  /*padding: 3px 10px 1px 3px;*/ /*removed by IBM*/
  padding: 3px 3px 1px 10px; /*Added by IBM*/
}

It seems bad, because with the current CSS, both the LTR and RTL editors have a 10px margin before the text. After your change, the RTL editor only has a 3px margin. This is inconsistent. (Tested on mac/chrome.)

About the new rule you added for IE:

.dj_ie .claro .dijitEditorRtl .dijitEditorIFrameContainer {
  padding: 3px 0px 1px 10px;
  margin-right: 0px;
  border: 0px solid #d3d3d3;

It's too general because it affects IE9 too, when IE9 is already working fine. But I will modify it to only affect IE6-8.

[bill]

In [29893]:

Fix right margin in Editor in RTL mode on IE6, IE7, IE8, based on patch from Hossam Katory (IBM, CCLA), thanks, fixes #15979.