Opened 7 years ago

Closed 7 years ago

#3940 closed Bug (fixed)

list operation doesn't stop at table

Reported by: garry.yao Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

Reproducing Procedures

  1. Open the attached example page.
  2. Select the table cell content, and click 'Create/Remove Bullted List'.
    • Actual Result:
      • list
        list in table
    • Expected Result:
      1. list
        • list in table

Attachments (4)

3940_TC.html (1.9 KB) - added by garry.yao 7 years ago.
Sample page for reproducing
3940.patch (3.2 KB) - added by tobiasz.cudnik 7 years ago.
3940_2.patch (3.3 KB) - added by tobiasz.cudnik 7 years ago.
3940_3.patch (3.5 KB) - added by tobiasz.cudnik 7 years ago.

Download all attachments as: .zip

Change History (13)

Changed 7 years ago by garry.yao

Sample page for reproducing

comment:1 Changed 7 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from new to assigned

Changed 7 years ago by tobiasz.cudnik

comment:2 Changed 7 years ago by tobiasz.cudnik

  • Keywords Review? added

This patch forbids to merge cell elements with table-wrapping list. It's also possible to create list selecting multiple block and table cells.

comment:3 Changed 7 years ago by garry.yao

  • Keywords Review- added; Review? removed

We should not target only the specific <td> tag on line L421:

if ( block.getName() != 'td' )

instead we could have:

if ( blockLimit.getName() != 'body' )

comment:4 Changed 7 years ago by tobiasz.cudnik

It doesn't perform so well with blockLimit check if "body". Video TC.

comment:5 Changed 7 years ago by garry.yao

Thanks for the video TC, I haven't test it, just want to explain my point, I mean we could group by blockLimit prior to grouping by ancestor list, and please test the patch carefully with enterkey, indent etc.

Changed 7 years ago by tobiasz.cudnik

comment:6 Changed 7 years ago by tobiasz.cudnik

Now condition checks if blockLimit is a body or block is nested list element and then allows to grouping by list ancestor.

What's missing i think is turning off list button ON state when focusing non-list inside a table which is inside a list.

One issue is closely releted list nested inside a table from indent plugin. When clicking enter second time to create new list element, JS error is thrown and bookmarks can be found in markup. Although this issue is not an regression and exists in trunk.

Changed 7 years ago by tobiasz.cudnik

comment:7 Changed 7 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

Garry extended the patch by providing toolbar button state changing and correcting indent plugin issue.

comment:8 Changed 7 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:9 Changed 7 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [3914].

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