Opened 14 years ago

Closed 14 years ago

Last modified 14 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 Minnekanti Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.4
Component: Core : Lists Version: 3.0.2
Keywords: IBM Confirmed Review+ Cc: Damian, joek

Description

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 14 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added
Milestone: CKEditor 3.3CKEditor 3.4
Version: 3.23.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 14 years ago by Garry Yao

Attachment: 5435.patch added

comment:2 Changed 14 years ago by Garry Yao

Component: GeneralCore : Styles
Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

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 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 14 years ago by Frederico Caldeira Knabben

Component: Core : StylesCore : Lists

comment:5 Changed 14 years ago by Tobiasz Cudnik

Resolution: fixed
Status: assignedclosed

Fixed with [5670].

comment:6 Changed 14 years ago by Tobiasz Cudnik

Committed into correct branch with [5694].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy