Ticket #3856 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Pressing the source toolbar button doesn't focus right

Reported by: jmraker Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.1
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

http://ecgdev.com/ajax1/test.html

Actual results in firefox: When I press the edit source button in the toolbar, the blur event is called and the focus event isn't called again until the editor is out of "edit source" mode. On focus the CKEditor should have a blue box around it. On blur it's gone. When clicking on the "edit source" button the blue border is gone after 1 or more clicks (usually one). When it's in "edit source" mode the border doesn't come back no matter where I click in the editor.

Actual results in IE8: The problem is reversed, where the editor's border stays blue and doesn't go away until it's out of edit source mode.

Expected results: The editor should call the focus/blur events in edit source mode so the editor's border is blue when focused, and not blue when not focused.

http://ecgdev.com/ajax1/test.html

Attachments

3856.patch (2.2 KB) - added by tobiasz.cudnik 5 years ago.

Change History

comment:1 Changed 5 years ago by jmraker

  • Milestone changed from CKEditor 3.x to CKEditor 3.0

comment:2 Changed 5 years ago by fredck

  • Milestone changed from CKEditor 3.0 to CKEditor 3.1

comment:3 Changed 5 years ago by tobiasz.cudnik

  • Status changed from new to assigned
  • Keywords Confirmed added
  • Owner set to tobiasz.cudnik

Changed 5 years ago by tobiasz.cudnik

comment:4 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added

Patch adds focus and blur events to source area mode. Additionally it fixes same events for webkit based browsers inside wysiwyg area mode (as they were broken).

Thanks for providing a test page.

comment:5 Changed 5 years ago by tobiasz.cudnik

  • Summary changed from ckeditor - Pressing the source toolbar button doesn't focus right to Pressing the source toolbar button doesn't focus right

comment:6 Changed 5 years ago by tobiasz.cudnik

#3996 has been marked as a DUP.

comment:7 Changed 5 years ago by garry.yao

  • Keywords Review+ added; Review? removed

Ticket Test added at : http://ckeditor.t/tt/3856/1.html.

comment:8 Changed 5 years ago by tobiasz.cudnik

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [4172].

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