Opened 14 years ago
Closed 14 years ago
#5681 closed Bug (wontfix)
'menubutton' force focus
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | UI : Context Menu | Version: | |
| Keywords: | Confirmed Review? | Cc: | WebSpellChecker.net |
Description
Currently the menu created by 'menubutton' plugin will always force focusing the editor on show and close, observed on SCAYT menu, the expected behavior is that focus should be set only when necessary.
Attachments (1)
Change History (7)
Changed 14 years ago by
| Attachment: | 5681.patch added |
|---|
comment:1 Changed 14 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:2 follow-up: 5 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
comment:3 Changed 14 years ago by
| Milestone: | CKEditor 3.3 |
|---|---|
| Resolution: | → wontfix |
| Status: | assigned → closed |
In fact, I don't see a problem of having menu buttons focusing the editor. After all, those buttons are related to the editor. Let's leave it this way for now, and check if we'll have troubles with it in the future.
comment:4 follow-up: 6 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|---|
| Resolution: | wontfix |
| Status: | closed → reopened |
This feature is required by SCAYT now: http://dev.fckeditor.net/ticket/5570#comment:11
comment:5 Changed 14 years ago by
| Milestone: | → CKEditor 3.3 |
|---|
Replying to fredck:
It may be simpler and safer to instead have a property in the CKEDITOR.plugins.contextMenu class, like CKEDITOR.plugins.contextMenu.editorFocus = true. It sounds less prone to error. It can default to true.
The 'editorFocus ' is a dynamic property depends on the opening moment of the menu, so it can't be a constructor field.
comment:6 Changed 14 years ago by
| Milestone: | CKEditor 3.3 |
|---|---|
| Resolution: | → wontfix |
| Status: | reopened → closed |
Replying to garry.yao:
This feature is required by SCAYT now
It should not be: comment:ticket:5570:13.
It may be simpler and safer to instead have a property in the CKEDITOR.plugins.contextMenu class, like CKEDITOR.plugins.contextMenu.editorFocus = true. It sounds less prone to error. It can default to true.