Opened 15 years ago
Closed 15 years ago
#5724 closed Bug (fixed)
[Firefox] Maximize one editor instance make other instances uneditable
| Reported by: | brooks | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | General | Version: | SVN (CKEditor) - OLD |
| Keywords: | Firefox Confirmed Review+ | Cc: |
Description (last modified by )
when there are multiple editor instances in one page, maximize one instance and then minimize it.this behavior will make other case uneditable any more.
Attachments (2)
Change History (10)
comment:1 Changed 15 years ago by
| Keywords: | Confirmed added |
|---|---|
| Milestone: | CKEditor 3.4 → CKEditor 3.3 |
Changed 15 years ago by
| Attachment: | 5724.patch added |
|---|
comment:2 Changed 15 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:3 Changed 15 years ago by
| Keywords: | Firefox added |
|---|
comment:4 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Keywords: | Review- added; Review? removed |
This code breaks if one of the editors is in source view. It must first check if the relative editor is in wysiwyg mode.
comment:5 Changed 15 years ago by
| Description: | modified (diff) |
|---|
Only checking one.document is not enough, because we may be on readonly mode (in the future).
Changed 15 years ago by
| Attachment: | 5724_2.patch added |
|---|
comment:6 follow-up: 7 Changed 15 years ago by
| Keywords: | Review? added; Review- removed |
|---|
What about checking editor.mode == 'wysiwyg'? In the future readonly view, editor.mode == 'readonly', probably.
comment:7 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Replying to garry.yao:
What about checking editor.mode == 'wysiwyg'? In the future readonly view, editor.mode == 'readonly', probably.
Brilliant idea! :P
Note: See
TracTickets for help on using
tickets.
This's an unfixed case of #5660.