Opened 10 years ago

Closed 10 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.

Find this issue on GitHub

Change History (5)

comment:1 Changed 10 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 10 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 10 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 10 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 10 years ago by Piotrek Koszuliński

Resolution: expired
Status: pendingclosed
Find this issue on GitHub
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