Ticket #5435 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

[[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

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

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

Change History

comment:1 Changed 5 years ago by fredck

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

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 5 years ago by garry.yao

comment:2 Changed 5 years ago by garry.yao

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

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 4 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 4 years ago by fredck

  • Component changed from Core : Styles to Core : Lists

comment:5 Changed 4 years ago by tobiasz.cudnik

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

Fixed with [5670].

comment:6 Changed 4 years ago by tobiasz.cudnik

Committed into correct branch with [5694].

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