Opened 14 years ago
Closed 11 years ago
#5722 closed Bug (wontfix)
[IE7] Paste into content RTL editor appears scrollbar
| Reported by: | Garry Yao | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | 3.4.1 |
| Keywords: | IE | Cc: |
Description
#5667 is still seen on IE6/7.
Attachments (3)
Change History (18)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
| Keywords: | Discussion added |
|---|
Any brilliant mind here to properly hide the paste bin in this case?
comment:3 Changed 14 years ago by
| Milestone: | CKEditor 3.3 → CKEditor 3.4 |
|---|
comment:4 Changed 14 years ago by
| Keywords: | HasPatch added; Discussion removed |
|---|
Using position: fixed resolves the problem.
Changed 14 years ago by
| Attachment: | 5722.patch added |
|---|
comment:5 Changed 14 years ago by
| Milestone: | CKEditor 3.4 → CKEditor 3.5 |
|---|
comment:6 Changed 14 years ago by
| Owner: | set to Sa'ar Zac Elias |
|---|---|
| Status: | confirmed → assigned |
Changed 14 years ago by
| Attachment: | 5722_2.patch added |
|---|
comment:7 Changed 14 years ago by
| Status: | assigned → review |
|---|
Since this the same bug also occurs in IE6, position:fixed would not work. So instead, just like in #5565, the pastebin is hidden to the top.
comment:8 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Wouldn't it be better to target this fix only to affected browsers ? Code to be removed looks like a good one, let's not loose it.
comment:9 Changed 14 years ago by
| Status: | review_failed → review |
|---|
Changed 14 years ago by
| Attachment: | 5722_3.patch added |
|---|
comment:10 Changed 14 years ago by
| Status: | review → review_passed |
|---|---|
| Version: | SVN (CKEditor) - OLD → 3.4.1 (SVN - trunk) |
comment:11 Changed 14 years ago by
| Status: | review_passed → review_failed |
|---|
Sorry... after patch the browser scrolls up and down on paste.
comment:12 Changed 14 years ago by
| Owner: | Sa'ar Zac Elias deleted |
|---|---|
| Status: | review_failed → new |
I don't have any other solution for that.. Hopefully someone else has.
comment:13 Changed 14 years ago by
| Milestone: | CKEditor 3.4.1 |
|---|---|
| Status: | new → confirmed |
As this issue impact on a reduced number of users (IE6/7+RTL) and it doesn't have a critical impact (just the scrollbar quick flashing), I'm untargetting it, so we can deal with it later.
comment:14 Changed 14 years ago by
| Keywords: | HasPatch removed |
|---|
comment:15 Changed 11 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | confirmed → closed |
Cannot reproduce on IE8 and we do not support IE7 any more.
Perhaps this would result in a wonfix as right:-1000 is not recognized in a RTL container in IE6/7.