Opened 9 years ago

Closed 9 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)

6025.patch (451 bytes) - added by Tobiasz Cudnik 9 years ago.
6025_2.patch (736 bytes) - added by Tobiasz Cudnik 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4
Status: newconfirmed

This is much probably related to the contenteditable change, so the focus should not anymore go to "window", but to "body".

comment:2 Changed 9 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: confirmedassigned

Changed 9 years ago by Tobiasz Cudnik

Attachment: 6025.patch added

comment:3 Changed 9 years ago by Tobiasz Cudnik

Status: assignedreview

Patch works well on Opera 10.60 (linux build 6386).

comment:4 Changed 9 years ago by Frederico Caldeira Knabben

Status: reviewreview_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 9 years ago by Tobiasz Cudnik

Status: review_failedreview

I've created ticket for future unifications as #6154.

Changed 9 years ago by Tobiasz Cudnik

Attachment: 6025_2.patch added

comment:6 Changed 9 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:7 Changed 9 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5821].

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy