Opened 5 years ago

Closed 4 years ago

#5955 closed Bug (fixed)

when we have combo box as first element in the Tool bar Application landmark is taking focus to Tool bar instead of Editor body

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

Description

To reproduce the defect

  1. Try to customize the Tool bar in such a way that you have any of the Combo boxes(Styles,Paragraph Format,Font Name,Font Size) as the first element in the Tool bar.
  1. with JAWS On..Press Ctrl + Insert + ; to open landmarks dialog
  1. Select Application landmark in the dialog and press Tab to go to Application landmark

Expected Result:

Focus goes in to Editor body.

Actual Result:

Focus goes to Combo box in the Tool bar and when we press Tab or use Arrow keys focus goes to next combo box in the Tool bar and after the combo boxes,focus is going in to Editor body instead of next Tool bar icon.

This is affecting all out products,In all our products we have the the combo boxes as the first element in the Tool bar.

Attachments (2)

5955_debug.patch (1.6 KB) - added by tobiasz.cudnik 5 years ago.
5955.patch (847 bytes) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 5 years ago by tobiasz.cudnik

  • Keywords Confirmed added

comment:2 Changed 5 years ago by fredck

  • Milestone changed from CKEditor 3.4 to CKEditor 3.5

comment:3 Changed 5 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from confirmed to assigned

Changed 5 years ago by tobiasz.cudnik

comment:4 Changed 5 years ago by tobiasz.cudnik

It seems JAWS doesn't push focus in the right way. I've attached patch for reproduction, dedicated for IE.

This situation is also happening for default toolbar setting. From my point of view, if we can't control the focus (the event isn't fired), then we can't fix it. This case needs deeper investigation.

comment:5 Changed 5 years ago by tobiasz.cudnik

  • Status changed from assigned to pending

comment:6 Changed 5 years ago by fredck

It would be better to have an opinion from Freedom Scientific.

comment:7 Changed 5 years ago by tobiasz.cudnik

I'm not sure if this isn't somehow related with focus issues we're facing in #6192...

Changed 5 years ago by garry.yao

comment:8 Changed 5 years ago by garry.yao

  • Milestone CKEditor 3.4.2 deleted

For me it's an unpunished of #3052.

comment:9 Changed 4 years ago by garry.yao

  • Cc changed from damo,joek to damo, joek
  • Owner changed from tobiasz.cudnik to garry.yao
  • Status changed from pending to review
  • Version changed from 3.3 to 3.0

comment:10 Changed 4 years ago by fredck

  • Status changed from review to review_passed

Just in case,it's better to follow the button code, checking is instance.onfocus is defined before calling it. Please do so on commit.

comment:11 Changed 4 years ago by fredck

  • Milestone set to CKEditor 3.6.1

comment:12 Changed 4 years ago by garry.yao

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

Fixed with [6945].

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