Opened 6 years ago

Last modified 6 years ago

#10607 confirmed Bug

Remove "indentlist" require from "list"

Reported by: Frederico Caldeira Knabben Owned by:
Priority: Normal Milestone:
Component: Core : Lists Version:
Keywords: Cc:

Description

This is a followup for ticket:10599#comment:3.

Ideally, the "list" plugin should not require "indentlist".

The problem is that there is a execCommand( 'outdent' ) call in the list plugin code, which makes this require necessary.

Additionally, we have removed some tests with (edbb565 @tests), because they were not compatible with this require. If we fix this, we should revert those tests back.

Change History (3)

comment:1 Changed 6 years ago by Frederico Caldeira Knabben

I have reverted edbb565 and pushed t/10607 @tests. In fact, it produces reds here:
http://ckeditor4.t/dt/plugins/indent/indent.html

comment:2 Changed 6 years ago by Frederico Caldeira Knabben

Status: newconfirmed

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

I had to fix one more thing in tests: see commit 49e3182.

It will be possible to revert this after fixing this ticket.

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