Opened 7 years ago

Closed 6 years ago

#5239 closed Bug (fixed)

Behaviour of Menus in Toolbar is Inconsistent

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.1
Component: UI : Toolbar Version: 3.1
Keywords: IBM Cc: damo, joek

Description

To reproduce the defect

For any Menu like Font Name, Font Size,Text Colour,Background Colour, Styles or Paragraph Format when we press Esc the Menu is closed and the Focus stays on the same Menu in the Toolbar

But for Spell Check as You Type Menu when we press Esc the Menu is closed and the Focus is going in to the body of the Editor.

Attachments (1)

5239.patch (421 bytes) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by fredck

  • Component changed from General to UI : Toolbar
  • Keywords Confirmed added
  • Milestone set to CKEditor 3.x

comment:2 Changed 6 years ago by fredck

  • Milestone CKEditor 3.x deleted

Milestone CKEditor 3.x deleted

comment:3 Changed 6 years ago by damo

We need to revisit this ticket.

We should have consistent behaviour of core UI components like the menu button. We use the menu button for features other than spell check.

comment:4 Changed 6 years ago by garry.yao

  • Cc changed from damo,joek to damo, joek
  • Component changed from UI : Toolbar to UI : Spell Checker

#6247 has fixed the generic focus returning logic, the rest to do is to update the focus management in SCAYT plugin.

Changed 6 years ago by garry.yao

comment:5 Changed 6 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

Oops, SCAYT is not the one to blame as there's still a legacy focus manipulation in menu plugin.

comment:6 Changed 6 years ago by fredck

  • Status changed from review to review_passed

comment:7 Changed 6 years ago by fredck

  • Component changed from UI : Spell Checker to UI : Toolbar
  • Milestone set to CKEditor 3.6.1

comment:8 Changed 6 years ago by garry.yao

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

Fixed with [6927].

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