Opened 14 years ago
Last modified 14 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 14 years ago by
Milestone: | CKEditor 3.4.3 → CKEditor 3.5.1 |
---|
comment:2 Changed 14 years ago by
comment:3 Changed 14 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 14 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 14 years ago by
Attachment: | 6615.patch added |
---|
comment:5 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:6 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6322].
comment:7 Changed 14 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:8 Changed 14 years ago by
Status: | reopened → confirmed |
---|
We decide to revert the fix with [6388].
comment:9 Changed 14 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 14 years ago by
Attachment: | 6615_2.patch added |
---|
comment:10 Changed 14 years ago by
Status: | confirmed → review |
---|
comment:11 Changed 14 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 14 years ago by
Attachment: | 6615_3.patch added |
---|
comment:12 Changed 14 years ago by
Status: | review_failed → review |
---|
New patch addresses the selection lost issue.
comment:13 Changed 14 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 14 years ago by
Attachment: | 6615_4.patch added |
---|
comment:14 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:15 Changed 14 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 14 years ago by
Milestone: | CKEditor 3.5.3 |
---|
Under IE7 the column pillar appears without clicking down mouse button.