Opened 7 years ago

Closed 7 years ago

#5975 closed Bug (fixed)

page-break should have proper Alt Text instead of Unknown object. so that JAWS reads it properly

Reported by: Satya Minnekanti Owned by: Tobiasz Cudnik
Priority: Normal Milestone: CKEditor 3.4.1
Component: General Version: 3.3
Keywords: IBM Cc: Damian, joek

Description

To reproduce the defect:

  1. with JAWS on, open Ajax sample.
  1. Type some text,Navigate to Tool bar by pressing Alt + F10.
  1. Navigate to Insert Page break icon and press Enter. see that Page break is inserted in the Editor.

See how JAWS reads the Page break..

JAWS is reading it as "Unknown object graphic".

if we give some proper name like "Page break".JAWS will read it properly and it will improve Accessibility and JAWS user can confirm that page break is inserted in Editor body.

Attachments (3)

5975.patch (1.9 KB) - added by Tobiasz Cudnik 7 years ago.
5975_2.patch (1.8 KB) - added by Tobiasz Cudnik 7 years ago.
5975_3.patch (1.4 KB) - added by Tobiasz Cudnik 7 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

comment:2 Changed 7 years ago by Charlie

I don't understand why it does that? (I don't have JAWS). It seems to have proper alt text though...

comment:3 Changed 7 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

Changed 7 years ago by Tobiasz Cudnik

Attachment: 5975.patch added

comment:4 Changed 7 years ago by Tobiasz Cudnik

Status: assignedreview

comment:5 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

The patch should be updated to the current markup.

Changed 7 years ago by Tobiasz Cudnik

Attachment: 5975_2.patch added

comment:6 Changed 7 years ago by Tobiasz Cudnik

Status: review_failedreview

comment:7 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

I don't think we should have the alt and the aria-label attributes in the output.

Changed 7 years ago by Tobiasz Cudnik

Attachment: 5975_3.patch added

comment:8 Changed 7 years ago by Tobiasz Cudnik

Status: review_failedreview

comment:9 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:10 Changed 7 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5867].

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