Opened 14 years ago
Closed 14 years ago
#6025 closed Bug (fixed)
Focus is lost after applying commands in Opera
| Reported by: | Sa'ar Zac Elias | Owned by: | Tobiasz Cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4 |
| Component: | General | Version: | 3.3.1 |
| Keywords: | Opera | Cc: |
Description
Steps to reproduce
- Open a sample and clear the document.
- Put the cursor in the editor.
- Apply a command.
Notice that the focus is lost.
Attachments (2)
Change History (9)
comment:1 Changed 14 years ago by
| Milestone: | → CKEditor 3.4 |
|---|---|
| Status: | new → confirmed |
comment:2 Changed 14 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | confirmed → assigned |
Changed 14 years ago by
| Attachment: | 6025.patch added |
|---|
comment:3 Changed 14 years ago by
| Status: | assigned → review |
|---|
Patch works well on Opera 10.60 (linux build 6386).
comment:4 Changed 14 years ago by
| Status: | review → review_failed |
|---|
- It's clear that the code is inside a if ( editor.window ) block, so the check should be changed to if ( editor.document ).
- In any case, as we're in Beta stabilization and the issue affects Opera only, let's have the path only for that browser, leaving it untouched for others. Then, after commit, let's open a ticket to simplify the code right after the 3.4 release.
comment:5 Changed 14 years ago by
| Status: | review_failed → review |
|---|
I've created ticket for future unifications as #6154.
Changed 14 years ago by
| Attachment: | 6025_2.patch added |
|---|
comment:6 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:7 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5821].
Note: See
TracTickets for help on using
tickets.
This is much probably related to the contenteditable change, so the focus should not anymore go to "window", but to "body".