Opened 16 years ago
Closed 16 years ago
#3174 closed Bug (fixed)
CKEditor 3.0 - Context Menu is not displayed correctly
| Reported by: | Senthil | Owned by: | Martin Kou |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | |
| Keywords: | IE Oracle Confirmed Review+ | Cc: | Senthil |
Description
Screen shot attached
Attachments (8)
Change History (26)
Changed 16 years ago by
| Attachment: | Context Menu.JPG added |
|---|
comment:1 Changed 16 years ago by
| Keywords: | Pending added |
|---|
comment:2 Changed 16 years ago by
| Keywords: | Confirmed added; Pending removed |
|---|
It is probably IE 6.0.
In FF 2.0 context menu also doesn't work.
comment:3 Changed 16 years ago by
| Keywords: | Firefox added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
Only reproduceable with FF 2.0.
Changed 16 years ago by
| Attachment: | 3174.patch added |
|---|
comment:4 Changed 16 years ago by
| Keywords: | Review? added |
|---|
The proposed patch introduce a simply way to fix the urgent issue of raising error, while a more graceful solution could be found later after fixing all other panel frame problem like #3175.
Changed 16 years ago by
| Attachment: | 3174_2.patch added |
|---|
Changed 16 years ago by
| Attachment: | 3174_3.patch added |
|---|
comment:7 Changed 16 years ago by
| Priority: | High → Normal |
|---|
Changed 16 years ago by
| Attachment: | Picture 7.png added |
|---|
comment:9 Changed 16 years ago by
| Owner: | changed from Garry Yao to Martin Kou |
|---|---|
| Status: | assigned → new |
comment:10 Changed 16 years ago by
IE7 needs some time to refresh the scrollHeight property of the popup menu's element.
Changed 16 years ago by
| Attachment: | 3174_4.patch added |
|---|
comment:11 Changed 16 years ago by
The delay needed in IE7 doesn't seem to have to do with CSS loading - everything from editor.css is already loaded by the time IE7 is trying to open the popup menu.
comment:12 Changed 16 years ago by
| Keywords: | IE added |
|---|
More problems were found on IE6:
- Sub menu arrow icon was doubled;
- Sub menu is broken when too much text of a sub menu item.
comment:13 Changed 16 years ago by
| Keywords: | Review- added; Firefox Review? removed |
|---|
In FF2, it WFM. I'm not sure those fixes are really needed, so if you can confirm it, please remove it from the patch.
It also WFM in IE7, but Martin states that he's still able to randomly reproduce it. So we could even consider a patch just for that.
Changed 16 years ago by
| Attachment: | Picture 2.png added |
|---|
comment:14 Changed 16 years ago by
It is still reproducible with today's SVN snapshot. It's a random bug, though.
comment:15 Changed 16 years ago by
I've found the follow method would make the bug easier to reproduce:
- Connect to a PC with IE7 or IE8 via Remote Desktop.
- Open replacebyclass.html in IE7 or IE8 of the remote machine.
- Right click on the "CKEditor" link in the editing area.
- If you're lucky, the menu will have a wrong size.
- If you're unlucky, refresh the page and try again.
The success rate varies. Sometimes I can have 5 or 6 successes in a row, but other times I can try for something like 40 times and it wouldn't happen once.
Changed 16 years ago by
| Attachment: | 3174_5.patch added |
|---|
comment:16 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
I've stripped away the FF2 part of the fix which is no longer needed.
comment:17 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
i was not able to reproduce it in any way, but it's evident that you have this problem. I'll just trust on your patch.
comment:18 Changed 16 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Fixed with [3482].
Click here for more info about our SVN system.
What's your browser version? I've tested it with IE7 and FF3, in both quirks and standards mode and it works well.