Ticket #4594 (closed Bug: fixed)
Context menu goes off-screen when mouse is at right had side of screen.
| Reported by: | JoeK | Owned by: | garry.yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | 3.0.1 |
| Keywords: | IBM Confirmed Review+ | Cc: | damian.chojna@…, pomu@… |
Description
Create an editor instance that spans the with of the screen. Move the mouse to the right hand side of the editor and right click. Observe the context menu is partially rendered off screen. See attached screen shot. The context menu should be rendered to the left of the mouse.
Scroll bars are rendered so it can be accessed if the user moves to the right.
Attachments
Change History
Changed 4 years ago by JoeK
- Attachment CKEditor Context Menu.bmp added
comment:1 Changed 4 years ago by pomu0325
- Cc pomu@… added
I faced the same problem and I did a workaround with this patch. Tested with IE8, FF3 and Chrome, only in 'ltr' situation.
comment:3 Changed 4 years ago by garry.yao
- Status changed from new to assigned
- Owner set to garry.yao
- Keywords Review? added; HasPatch removed
Implement full off-screen support for context menu inspired by pomu's patch.
comment:4 Changed 4 years ago by garry.yao
As discussed with Fred, we may even come out a "just work" solution which cascade float panels for off-screen, which is more implementation lightweight than the previous patch.
Ticket Test added at :
http://ckeditor.t/tt/4594/1.html.
comment:5 Changed 4 years ago by fredck
- Keywords Review- added; Review? removed
It looks like page scrolling impacts on the calculations. In both IE and FF, the panel opens to the top if you scroll down the page and opens the context menu, even if there is room for it at the bottom of the cursor.
comment:6 Changed 4 years ago by garry.yao
- Keywords Review? added; Review- removed
Nice catch indeed.
comment:7 Changed 4 years ago by fredck
- Keywords Review- added; Review? removed
After patching, in FF:
- Load a sample.
- Click the color selector.
- With the color selector opened, click any of the combos.
The color selector remains opened. It happens only on first opening, with all combinations of panels opening order, including the context menu.
In IE instead, the panels don't show on first click.
comment:8 Changed 4 years ago by garry.yao
- Keywords Review? added; Review- removed
The activation function is been placed at wrong place in previous patch.
comment:9 Changed 3 years ago by fredck
- Keywords Review- added; Review? removed
No change in IE. The panels are still not showing on first click.
comment:11 Changed 3 years ago by fredck
- Keywords Review- added; Review? removed
I confirm that in IE8 the panels don't appear on first click.
comment:14 Changed 3 years ago by garry.yao
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [4606].
comment:15 Changed 3 years ago by damo
- Status changed from closed to reopened
- Resolution fixed deleted
This solution partially fixes the problem. The sub menu displays over the main menu, making it impossible to navigate around the context menu properly. The sub menu should really display to the side of the main menu.
See attached screenshot: SubmenuOverlap.gif
comment:16 Changed 3 years ago by garry.yao
- Status changed from reopened to closed
- Resolution set to fixed
The sub menu displays over the main menu...
Please refer to previous comment as our decision for it, feel free to open a new ticket for the feature you're suggesting though.
Off-screen context menu screen shot