Opened 7 years ago

Closed 7 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)

2009-07-24-102150_539x95_scrot.png (4.9 KB) - added by tobiasz.cudnik 7 years ago.
4096.patch (1.2 KB) - added by tobiasz.cudnik 7 years ago.
4096_2.patch (1.3 KB) - added by tobiasz.cudnik 7 years ago.
4096_3.patch (1.3 KB) - added by tobiasz.cudnik 7 years ago.

Download all attachments as: .zip

Change History (13)

Changed 7 years ago by tobiasz.cudnik

comment:1 Changed 7 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from new to assigned

comment:2 Changed 7 years ago by garry.yao

Confirmed on all skins except Kama.

Changed 7 years ago by tobiasz.cudnik

comment:3 Changed 7 years ago by tobiasz.cudnik

  • Keywords Review? added

comment:4 Changed 7 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 7 years ago by tobiasz.cudnik

comment:5 Changed 7 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 7 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 7 years ago by tobiasz.cudnik

comment:7 Changed 7 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

comment:8 Changed 7 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:9 Changed 7 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.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy