Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#13507 closed Bug (fixed)

Existing text in widget editables can't be edited in Firefox (39)

Reported by: Tomas Bures Owned by:
Priority: Normal Milestone:
Component: UI : Widgets Version: 4.5.1
Keywords: Cc:

Description

Steps to reproduce (in Firefox 39, Windows 8)

This seems to be connected with the problem that the cursor can be placed only at the beginning, not in the middle of the text.

This bug seems to be limited only to Firefox. In IE and Chrome, it seems to work.

Attachments (1)

13507-ff40-nestededitable-issue.mp4 (419.6 KB) - added by Olek Nowodziński 10 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 Changed 10 years ago by Piotrek Koszuliński

Status: newpending

Do you by any chance asked this question: http://stackoverflow.com/questions/31257123/unexpected-behaviour-of-nested-contenteditable-divs-in-firefox?

If not, then this must not be a coincidence that in 14 hours I saw two bug reports about it. It may be a serious regression in the newest version of Firefox ;/.

comment:2 Changed 10 years ago by Tomas Bures

The guy asking at stackoverflow was someone else. However, he/she seems to describe the same behavior.

comment:3 Changed 10 years ago by Olek Nowodziński

comment:4 Changed 10 years ago by Olek Nowodziński

Status: pendingconfirmed

comment:5 Changed 10 years ago by Piotrek Koszuliński

The FF team responded immediately so there's a chance that this issue will be fixed very soon :).

Last edited 10 years ago by Piotrek Koszuliński (previous) (diff)

comment:6 in reply to:  5 Changed 10 years ago by Sean D Kendle

Replying to Reinmar:

The FF team responded immediately so there's a chance that this issue will be fixed very soon :).

I'm guessing there hasn't been an update on this? I just spent an hour scouring my code thinking I had messed something up, and updated to the latest CKEditor (4.5.2) before I found that this was a known bug. I can confirm that IE and Chrome do, indeed, allow editing of the "editables".

What's strange is that I made a quick and dirty test page for a div with "contenteditable=true" and it works fine in FF. I wonder what the difference is?

Last edited 10 years ago by Sean D Kendle (previous) (diff)

comment:7 Changed 10 years ago by Piotrek Koszuliński

Single contenteditable element works. Nested does not work. See http://jsfiddle.net/oleq/hfLfjd55/

And yes... there's no update. First, the Firefox team wanted to release a patch in 39.0.1, then they commented that it will take more time:

The issue will be fixed on August 11 with Firefox 40, but if you are deploying Firefox in your institution, consider using the stable Extended Support Release (ESR) instead: https://www.mozilla.org/en-US/firefox/organizations/

But it's August 7 now and it's still open, so I don't think they'll manage to close it...

comment:8 Changed 10 years ago by Piotrek Koszuliński

But it's August 7 now and it's still open, so I don't think they'll manage to close it...

I may be wrong. I see here https://bugzilla.mozilla.org/show_bug.cgi?id=1132768 that this patch has been reverted. So if Firefox 40 will ship without it, then it should be back to normal. Perhaps that's why they don't have to hurry with the bug that we reported, using a chance to fix it neatly.

comment:9 in reply to:  8 Changed 10 years ago by Sean D Kendle

Thank you for your attention to this matter!

comment:10 Changed 10 years ago by Jakub Ś

#13652 was marked as duplicate.

comment:11 Changed 10 years ago by Piotrek Koszuliński

I confirm that it works fine again in Firefox 40.

comment:12 in reply to:  11 Changed 10 years ago by Olek Nowodziński

Replying to Reinmar:

I confirm that it works fine again in Firefox 40.

Not exactly.

  1. http://ckeditor.com/demo#widgets
  2. Focus nested editable of simplebox widget.
  3. CTRL+A
  4. CTRL+X
  5. CTRL+V
  6. Left arrow.

Expected: Second list item^,

Actual: Selection jumps between lines and nested editables.

Last edited 10 years ago by Olek Nowodziński (previous) (diff)

Changed 10 years ago by Olek Nowodziński

comment:13 Changed 10 years ago by Piotrek Koszuliński

There were some issues before, so this may not be a regression.

comment:14 Changed 10 years ago by Jakub Ś

Resolution: fixed
Status: confirmedclosed

@a.nowodzinski your findings are describing ticket #10988.

I have tried you TC with Image2 and results were the same as @m.lewandowski results for image2.

I have linked your comment to that ticket and closing this one as fixed.

Last edited 10 years ago by Olek Nowodziński (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy