Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#7316 closed Bug (fixed)

[FF] Click on paste button create wrong linebreak

Reported by: garry.yao Owned by: garry.yao
Priority: High Milestone: CKEditor 3.5.3
Component: Core : Pasting Version: 3.5.2
Keywords: Firefox IBM Cc: satya_minnekanti@…

Description

  1. Place cursor among text;
  2. Click on 'Paste' or 'Paste from Word' button;
  • Actual Result: Line break inserted at the position.

Attachments (2)

7316.patch (1.6 KB) - added by garry.yao 6 years ago.
7316_2.patch (1008 bytes) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (17)

Changed 6 years ago by garry.yao

comment:1 Changed 6 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to review

Regression of [6373], where pastebin was affected by bogus br.

comment:2 Changed 6 years ago by satya

  • Cc satya_minnekanti@… added

comment:3 Changed 6 years ago by Saare

#7331 is a dup.

comment:4 Changed 6 years ago by wwalc

  • Keywords IBM added
  • Milestone set to CKEditor 3.5.3

comment:5 Changed 6 years ago by fredck

  • Status changed from review to review_failed
  • The new "beforeSelectionChange" event is not needed. It's enough to listen to "selectionChange" with higher priority, canceling the event.
  • The removeListener() call is missing the listener function to be removed.

Changed 6 years ago by garry.yao

comment:6 Changed 6 years ago by garry.yao

  • Status changed from review_failed to review

comment:7 Changed 6 years ago by Saare

  • Status changed from review to review_passed

comment:8 Changed 6 years ago by garry.yao

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

Fixed with [6556].

comment:9 Changed 6 years ago by jamescun

This is still reproducible on the latest nightly build on FF 3.6.15 and should be reopened.

comment:10 Changed 6 years ago by jamescun

This issue was not fixed as I can reproduce it on FF 3.6.6, 3.6.8 and 3.6.15. I would like this ticket to be reopened and this defect to be fixed. Thanks.

comment:11 Changed 6 years ago by wwalc

I have just tried to reproduce it in the trunk and on the nightly demo page and it works for me. Could you please check it again? Please make sure you have cleared browser's cache.

Note that I did manage to reproduce it on Rev 6555 samples but in recent revisions like Rev 6593 samples it works fine.

comment:12 Changed 6 years ago by jamescun

Sorry I failed to mention earlier that you must click ok on the Paste dialog to reproduce the defect outlined in the test case.

comment:13 Changed 6 years ago by wwalc

Please use FF 3.6.15 and open the following sample: Rev 6593 samples

  1. Select "some" (third word), press Ctrl+C
  2. Click between "You" and "are": "You |are"
  3. Press the "Paste" button
  4. In the "Paste" dialog press "Ctrl + V" (the cursor is blinking straight after "some", in the same line)
  5. Press "Ok" button
  6. The result is correct on my side:
    This is some <strong>sample text</strong>. You someare using (...)
    

Is it the same way how you reproduce it?

comment:14 Changed 6 years ago by jamescun

No I am reproducing it a different way.

  1. Open Rev 6593 samples in FF 3.6.15
  1. Click between "You" and "are": "You |are"
  1. Open the Paste dialog.
  1. Leave the paste dialog empty & click ok.

Result: Line break inserted between "You" and "are".

comment:15 Changed 6 years ago by wwalc

Confirmed and reported in #7368.

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