Opened 9 years ago

Closed 9 years ago

#6017 closed Bug (fixed)

tableresize plugin refactoring

Reported by: Frederico Caldeira Knabben Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.4
Component: General Version:
Keywords: Cc:

Description

We need to review a bit some aspects of the tableresize plugin. The current implementation still have issues:

  • Not compatible with IE6, IE7 and IE+Quirks (#5890).
  • It's not possible to resize the last column (#5891).
  • No support for RTL (#5995).

It's not possible to bring minor changes to the code to be able to fix all these issues. A major change is needed. I'm opening this ticket to handle all these at once.

Attachments (1)

6017.patch (18.8 KB) - added by Frederico Caldeira Knabben 9 years ago.

Download all attachments as: .zip

Change History (10)

Changed 9 years ago by Frederico Caldeira Knabben

Attachment: 6017.patch added

comment:1 Changed 9 years ago by Frederico Caldeira Knabben

Status: newreview

I'm reviewing the entire tableresize plugin with this patch, changing the logic in some places, simplifying things in others, and even clean up things a bit.

All the reported issues should be fixed.

comment:2 Changed 9 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

IE Quirks: hovering over a cell's border displays the resizer and not the resizing cursor.

comment:3 Changed 9 years ago by Sa'ar Zac Elias

Owner: changed from Frederico Caldeira Knabben to Sa'ar Zac Elias
Status: review_failedreview

Sorry, I didn't notice the comment before.

comment:4 Changed 9 years ago by Sa'ar Zac Elias

Status: reviewassigned

comment:5 Changed 9 years ago by Sa'ar Zac Elias

Owner: Sa'ar Zac Elias deleted
Status: assignednew

comment:6 Changed 9 years ago by Sa'ar Zac Elias

Owner: set to Sa'ar Zac Elias
Status: newreview

Fred please take over this ticket again..

comment:7 Changed 9 years ago by Frederico Caldeira Knabben

Owner: changed from Sa'ar Zac Elias to Frederico Caldeira Knabben

comment:8 Changed 9 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:9 Changed 9 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [5747].

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