Opened 4 years ago

Closed 4 years ago

#13391 closed Bug (expired)

double quote issue

Reported by: Amit Owned by:
Priority: Normal Milestone:
Component: Core : Output Data Version:
Keywords: Cc:

Description

when adding text like as <div style="width: 100%; height:30px; margin: 0px; padding: 10px; background: #669c22; color: #fff;>

i forgot to close quot(") of style then ckeditor not open.

Change History (5)

comment:1 Changed 4 years ago by Amit

a alert will be pop up with message given below.

A script on this page may be busy, or it may have stopped responding. You can stop the script now, open the script in the debugger, or let the script continue.

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

Status: newpending
Version: 4.5.0 (GitHub - major)

I can't reproduce it. Could you provide more details:

  1. On what browser it happens?
  2. Does it happen on http://ckeditor.com/demo.
  3. If not - a working sample.

comment:3 Changed 4 years ago by Jakub Ś

The message looks Firefox to me. I have seen them before.

I have tried demo and local editor full mode:

Works:

<div style="width: 100%; height:30px; margin: 0px; padding: 10px; background: #669c22; color: #fff;">abc</div>

HTML is removed completely:

<div style="width: 100%; height:30px; margin: 0px; padding: 10px; background: #669c22; color: #fff;>abc</div>
<div style="width: 100%; height:30px; margin: 0px; padding: 10px; background: #669c22; color: #fff;>

Removal is questionable for first example with only " missing but it is always better then error being thrown, or infinite loop which I have not observed.

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

@jswiderski: Missing quote is a pretty serious issue in HTML, so even browsers may be dropping the whole thing.

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

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