Opened 8 years ago
Closed 8 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 8 years ago by tobiasz.cudnik
comment:1 Changed 8 years ago by tobiasz.cudnik
- Owner set to tobiasz.cudnik
- Status changed from new to assigned
comment:2 Changed 8 years ago by garry.yao
Changed 8 years ago by tobiasz.cudnik
comment:3 Changed 8 years ago by tobiasz.cudnik
- Keywords Review? added
comment:4 Changed 8 years ago by fredck
- 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 8 years ago by tobiasz.cudnik
comment:5 Changed 8 years ago by tobiasz.cudnik
- 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 8 years ago by fredck
- 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 8 years ago by tobiasz.cudnik
comment:7 Changed 8 years ago by tobiasz.cudnik
- Keywords Review? added; Review- removed
comment:8 Changed 8 years ago by fredck
- Keywords Review+ added; Review? removed
comment:9 Changed 8 years ago by tobiasz.cudnik
- Resolution set to fixed
- Status changed from assigned to closed
Fixed with [3984].
Note: See
TracTickets for help on using
tickets.
Confirmed on all skins except Kama.