Opened 16 years ago
Closed 16 years ago
#4096 closed Bug (fixed)
Editor content area pushed down in RTL quirks
| Reported by: | Tobiasz Cudnik | Owned by: | Tobiasz Cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | |
| Keywords: | IE Confirmed Review+ | Cc: |
Description
Editor content area pushed down in RTL quirks. All IEs affected. Attaching screenshot.
Attachments (4)
Change History (13)
Changed 16 years ago by
| Attachment: | 2009-07-24-102150_539x95_scrot.png added |
|---|
comment:1 Changed 16 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | new → assigned |
comment:2 Changed 16 years ago by
Changed 16 years ago by
| Attachment: | 4096.patch added |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review? added |
|---|
comment:4 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
There is still a 1px space being left there.
I had better results by setting position:absolute, instead of changing font-size.
Changed 16 years ago by
| Attachment: | 4096_2.patch added |
|---|
comment:5 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
You're right. In second patch i've used position absolute and targeted this hack only for [IE, quirks, RTL].
I've done some tests with nesting editor inside scrollable area and it seems no problems caused by this change.
comment:6 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
I was able to reproduce it in IE+LTR+Quirks also. I've then remove the RTL check from the patch and it fix it also. I've tested it with all browsers and it looks good, so I think we could do it on LTR safely.
Changed 16 years ago by
| Attachment: | 4096_3.patch added |
|---|
comment:7 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:8 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
Confirmed on all skins except Kama.