Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#5435 closed Bug (fixed)

[[IE]] we can't start Numbered/Bulleted list in Tables by Clicking on Insert/Remove Numbers/Bullets Icon

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.4
Component: Core : Lists Version: 3.0.2
Keywords: IBM Confirmed Review+ Cc: damo, joek


To reproduce the defect

  1. Open Ajax sample.
  1. Insert a Table with some no of rows and Columns
  1. Go in to one of Table Cell and Click on Insert/Remove Numbers or Bullets Icon

Actual Result:

Nothing happens,even when we type the text in the Table Cell it just shows as a Normal Text but not as Numbered/Bulleted list item

Expected Result:

A new Numbered/Bulleted list should start depending up on the Insert/Remove Numbers/Bullets Icon that you click on.

I have tested against IE 6,7 & 8

This is a regression issue.

Attachments (1)

5435.patch (2.3 KB) - added by garry.yao 7 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 7 years ago by fredck

  • Keywords Confirmed added
  • Milestone changed from CKEditor 3.3 to CKEditor 3.4
  • Version changed from 3.2 to 3.0.2

This is not a recent regression. I can confirm it works with CKEditor 3.0.1 and the issue has been introduced with version 3.0.2.

Changed 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

  • Component changed from General to Core : Styles
  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

The patch proposes removing the buggy "empty range" checking logic when iterating over "noisy" ranges, in favor of our new range::shrink armor, it's supposed to be able to handle all the previous mentioned bug while has no misunderstanding on good range like the one in this ticket.

comment:3 Changed 7 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 7 years ago by fredck

  • Component changed from Core : Styles to Core : Lists

comment:5 Changed 7 years ago by tobiasz.cudnik

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

Fixed with [5670].

comment:6 Changed 7 years ago by tobiasz.cudnik

Committed into correct branch with [5694].

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