Opened 4 years ago

Closed 23 months ago

#13096 closed Bug (fixed)

Another CKEditor Exception in Firefox

Reported by: Alan Owned by: Mateusz Samsel
Priority: Normal Milestone:
Component: Core : Keystrokes Version: 4.0 Beta
Keywords: Firefox Cc:

Description

I filed bug #13020 a couple weeks ago explaining that we're seeing lots of CKEditor exceptions in our code. Here's a second one that we've been able to reproduce in jsfiddle. I will also attached a video.

To demonstrate the bug go to: http://jsfiddle.net/ftey46fc/14/

1) Type 'test' in the editor 2) Click to the left of 'test' between it and the border. About half

way works fine. (You can basically click anywhere in the 'padding' between the editor's content and the word e.g. you can click above it too.

3) Press the 'Delete' or 'Backspace' key.

The Video is viewable here: http://youtu.be/QNjrwfnpGwI

Video timeline:

0:00 - Show URL 0:08 - Show current console messages (e.g. that the exception has not happened yet) 0:14 - Show that we are in fact at the end of the console log (no hidden messages) 0:22 - Show Firefox version 0:32 - Type 'test' in the editor 0:39 - Click to the left of the word 'test' 0:42 - Show that the exception has not happened yet 0:46 - Press 'Delete' key 0:50 - Exception 'h is undefined' triggered

Attachments (1)

replacebycode4.html (2.2 KB) - added by Jakub Ś 4 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 4 years ago by Alan

Note: You can also press Enter instead of backspace/delete and generate a different 'elementPath(...) is null' exception. Of course, it's all really the same thing (the browser's current selection has no ranges).

comment:2 Changed 4 years ago by Jakub Ś

Keywords: exception CSS removed
Status: newconfirmed
Version: 4.4.74.0 Beta

Problem can be reproduced from CKEditor 4.0 beta and doesn't occur when you remove 'position: absolute;' from inserted CSS.

Changed 4 years ago by Jakub Ś

Attachment: replacebycode4.html added

comment:3 Changed 2 years ago by Mateusz Samsel

Owner: set to Mateusz Samsel
Status: confirmedassigned

comment:4 Changed 2 years ago by Mateusz Samsel

Status: assignedreview

comment:5 Changed 2 years ago by Mateusz Samsel

Enter key issue is treat as separate case and it's handled in ticket #16874.

comment:6 Changed 23 months ago by kkrzton

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