Opened 4 years ago

Closed 4 years ago

Last modified 4 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 4 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 Changed 4 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 4 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 4 years ago by Olek Nowodziński

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

Status: pendingconfirmed

comment:5 Changed 4 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 4 years ago by Piotrek Koszuliński (previous) (diff)

comment:6 in reply to:  5 Changed 4 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 4 years ago by Sean D Kendle (previous) (diff)

comment:7 Changed 4 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 4 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 4 years ago by Sean D Kendle

Thank you for your attention to this matter!

comment:10 Changed 4 years ago by Jakub Ś

#13652 was marked as duplicate.

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

I confirm that it works fine again in Firefox 40.

comment:12 in reply to:  11 Changed 4 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 4 years ago by Olek Nowodziński (previous) (diff)

Changed 4 years ago by Olek Nowodziński

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

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

comment:14 Changed 4 years ago by Jakub Ś

Resolution: fixed
Status: confirmedclosed

@a.nowodzinski your findings are describing ticket #19088

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.

Version 0, edited 4 years ago by Jakub Ś (next)
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy