Opened 13 years ago
Last modified 13 years ago
#6615 review_failed Bug
[IE] tableresize usability
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | 3.4 Beta |
| Keywords: | IE | Cc: |
Description
The changes of [5747] introduced the following problems in IE:
- Column resizer's position doesn't accurately reflect cell spaces, few pixels offset makes the resizer always appear before mouse is actually moved into;
- The cursor shape is inconsistent with other browsers, even composed of two sharps when moving along the visible region.
Attachments (4)
Change History (20)
comment:1 Changed 13 years ago by
| Milestone: | CKEditor 3.4.3 → CKEditor 3.5.1 |
|---|
comment:2 Changed 13 years ago by
comment:3 Changed 13 years ago by
| Status: | new → confirmed |
|---|
I can reproduce first & second TCs when gripping the resizer from near the cell border (could be done easily with bigger cellpadding value).
comment:4 Changed 13 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
By back-tracing I understand that the issue was originated from #5890, affects only below IE8.
Changed 13 years ago by
| Attachment: | 6615.patch added |
|---|
comment:5 Changed 13 years ago by
| Status: | review → review_passed |
|---|
comment:6 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6322].
comment:7 Changed 13 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:8 Changed 13 years ago by
| Status: | reopened → confirmed |
|---|
We decide to revert the fix with [6388].
comment:9 Changed 13 years ago by
| Milestone: | CKEditor 3.5.1 → CKEditor 3.5.2 |
|---|
The patch from #7083 seems to fix both reported problems, needs deeper testing though.
Changed 13 years ago by
| Attachment: | 6615_2.patch added |
|---|
comment:10 Changed 13 years ago by
| Status: | confirmed → review |
|---|
comment:11 Changed 13 years ago by
| Status: | review → review_failed |
|---|
IE7:
Create a table, put contents in first and second cells and select their contents using the mouse. Now click on the border between them and drag it a bit. Once one releases the grip, the selection is lost.
Changed 13 years ago by
| Attachment: | 6615_3.patch added |
|---|
comment:12 Changed 13 years ago by
| Status: | review_failed → review |
|---|
New patch addresses the selection lost issue.
comment:13 Changed 13 years ago by
| Status: | review → review_failed |
|---|
IE7 mode, create a table and put a list in the rightmost cell in the top row. Click and drag the border between the cells. Note that the resizer does not appear correctly.
Changed 13 years ago by
| Attachment: | 6615_4.patch added |
|---|
comment:14 Changed 13 years ago by
| Status: | review_failed → review |
|---|
comment:15 Changed 13 years ago by
| Status: | review → review_failed |
|---|
IE7 is still broken:
- Put the cursor at the top-rightmost cell. Click on the list button.
- Without changing the selection, hover over the border.
- See that helper does not appear.
- Now switch to source and then back to wysiwyg.
- Hover over the middle border.
- Note that the left side is "disappearing", while it's still intact in source.
Another issue: do as described in comment:11. Note that the focus is lost.
comment:16 Changed 13 years ago by
| Milestone: | CKEditor 3.5.3 |
|---|
Under IE7 the column pillar appears without clicking down mouse button.