Opened 7 years ago

Closed 7 years ago

#9601 closed Bug (fixed)

Support for config.disableReadonlyStyling is missed in v4

Reported by: Piotrek Koszuliński Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.0
Component: Core : Styles Version: 4.0
Keywords: Cc:

Description

There's a doc, but nothing else :).

Lack of this feature may be verified in placeholder plugin - in v3 placeholder may be bolded, in v4 no.

Change History (8)

comment:1 Changed 7 years ago by Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: newassigned

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

Status: assignedreview

Pushed t/9601.

comment:3 Changed 7 years ago by Garry Yao

Component: GeneralCore : Styles
Status: reviewreview_failed

Unlike v3, document could be shared by multiple inline editor instances, thus the configuration data must go onto editable instead.

comment:4 Changed 7 years ago by Frederico Caldeira Knabben

We talked about it yesterday and decided that we'll just port it as is because the possibility of having someone facing this issue is close to zero. We may change it later if we feel necessary.

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

I pushed t/9601 on tests and dev. This time config is passed by editable's custom data.

comment:6 Changed 7 years ago by Piotrek Koszuliński

Status: review_failedreview

comment:7 Changed 7 years ago by Garry Yao

Status: reviewreview_passed

comment:8 Changed 7 years ago by Garry Yao

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