Opened 8 years ago
Closed 8 years ago
#15979 closed defect (fixed)
extra space exists in mirrored Editor
Reported by: | ahmedasaleh | Owned by: | bill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.9 |
Component: | Editor | Version: | 1.8.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
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
Attachments (2)
Change History (6)
Changed 8 years ago by
Attachment: | extra_spaces.JPG added |
---|
comment:2 Changed 8 years ago by
Milestone: | tbd → 1.9 |
---|---|
Owner: | set to bill |
Status: | new → assigned |
comment:3 Changed 8 years ago by
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.
Note: See
TracTickets for help on using
tickets.
Extra spaces are added in mirrored dijit.Editor