Ticket #5955 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

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

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

Change History

comment:1 Changed 4 years ago by tobiasz.cudnik

  • Keywords Confirmed added

comment:2 Changed 4 years ago by fredck

  • Milestone changed from CKEditor 3.4 to CKEditor 3.5

comment:3 Changed 4 years ago by tobiasz.cudnik

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

Changed 4 years ago by tobiasz.cudnik

comment:4 Changed 4 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 4 years ago by tobiasz.cudnik

  • Status changed from assigned to pending

comment:6 Changed 4 years ago by fredck

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

comment:7 Changed 4 years ago by tobiasz.cudnik

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

Changed 3 years ago by garry.yao

comment:8 Changed 3 years ago by garry.yao

  • Milestone CKEditor 3.4.2 deleted

For me it's an unpunished of #3052.

comment:9 Changed 3 years ago by garry.yao

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

comment:10 Changed 3 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 3 years ago by fredck

  • Milestone set to CKEditor 3.6.1

comment:12 Changed 3 years ago by garry.yao

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

Fixed with [6945].

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