Opened 9 years ago

Closed 9 years ago

#7694 closed Bug (fixed)

Block style on read-only

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.4
Component: Core : Read-only Version: 3.5
Keywords: Cc:

Description

Apply block styles (e.g. Format) on the following content and selection:

<p>paragraph [one</p>
<div contenteditable="false">none editable block</div>
<p>paragraph] two</p>

Actual Result: the read-only block will be affected.

Attachments (2)

7694.patch (1.7 KB) - added by Garry Yao 9 years ago.
7694_2.patch (900 bytes) - added by Garry Yao 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by Jakub Ś

Status: newconfirmed
Version: 3.43.4.3

It seems that it has been working in CKEditor versions 3.4 - 3.4.3 (inclusive).

This issue concerns all browsers

comment:2 Changed 9 years ago by Garry Yao

Version: 3.4.33.5

Regression of [6128].

Changed 9 years ago by Garry Yao

Attachment: 7694.patch added

comment:3 Changed 9 years ago by Garry Yao

Component: Core : StylesCore : Read-only
Owner: set to Garry Yao
Status: confirmedreview

Changed 9 years ago by Garry Yao

Attachment: 7694_2.patch added

comment:4 Changed 9 years ago by Garry Yao

A simplified patch based on Fred's suggestion.

comment:5 Changed 9 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:6 Changed 9 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.5.4

comment:7 Changed 9 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6820].

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