Opened 14 years ago
Closed 14 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)
Change History (10)
Changed 14 years ago by
Attachment: | 6017.patch added |
---|
comment:1 Changed 14 years ago by
Status: | new → review |
---|
comment:2 Changed 14 years ago by
Status: | review → review_failed |
---|
IE Quirks: hovering over a cell's border displays the resizer and not the resizing cursor.
comment:3 Changed 14 years ago by
Owner: | changed from Frederico Caldeira Knabben to Sa'ar Zac Elias |
---|---|
Status: | review_failed → review |
Sorry, I didn't notice the comment before.
comment:4 Changed 14 years ago by
Status: | review → assigned |
---|
comment:5 Changed 14 years ago by
Owner: | Sa'ar Zac Elias deleted |
---|---|
Status: | assigned → new |
comment:6 Changed 14 years ago by
Owner: | set to Sa'ar Zac Elias |
---|---|
Status: | new → review |
Fred please take over this ticket again..
comment:7 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Frederico Caldeira Knabben |
---|
comment:8 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:9 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [5747].
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.