Ticket #5667 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

Pasting in an RTL page causes the editor to scroll erratically

Reported by: Saare Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Pasting Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

Though reproducible in all browsers, the bug is most noticable in IE.

Steps to reproduce

  • Open a sample page in RTL mode.
  • Paste some content inside the editor.

Notice that the scrollbar behaves erratically. As long as you paste more data or faster, longer the time the scrollbar appears.
This bug does not occur in 3.2.1.

Attachments

5667.patch (1.1 KB) - added by garry.yao 4 years ago.

Change History

comment:1 Changed 4 years ago by garry.yao

  • Milestone CKEditor 3.3 deleted

Can you please provide more details than 'erratically'?

comment:2 Changed 4 years ago by Saare

  • Milestone set to CKEditor 3.3

Sorry, I mistakenly thought the cause to this problem is the RTL page, but it seems that the cause is the contentsLangDirection : 'rtl' definition.
If you paste you'll notice that the horizontal scrollbar is jumping to the left. You will also notice that there is an element selected and focused.
Regression of [5224], so targeting to 3.3.

Changed 4 years ago by garry.yao

comment:3 Changed 4 years ago by garry.yao

  • Owner set to garry.yao
  • Keywords Confirmed Review? added
  • Status changed from new to assigned

comment:4 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:5 Changed 4 years ago by garry.yao

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [5493].

comment:6 Changed 3 years ago by wizztjh

It still cause problem when the ckeditor is on the middle of scrollbar. After delete the

pastebin.setStyle( this.config.contentsLangDirection == 'ltr' ? 'left' : 'right', '-1000px' );

it is fine. Why we need that line of code?

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy