Opened 10 years ago

Closed 10 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 10 years ago.
5975_2.patch (1.8 KB) - added by Tobiasz Cudnik 10 years ago.
5975_3.patch (1.4 KB) - added by Tobiasz Cudnik 10 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

comment:2 Changed 10 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 10 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

Changed 10 years ago by Tobiasz Cudnik

Attachment: 5975.patch added

comment:4 Changed 10 years ago by Tobiasz Cudnik

Status: assignedreview

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

Status: reviewreview_failed

The patch should be updated to the current markup.

Changed 10 years ago by Tobiasz Cudnik

Attachment: 5975_2.patch added

comment:6 Changed 10 years ago by Tobiasz Cudnik

Status: review_failedreview

comment:7 Changed 10 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 10 years ago by Tobiasz Cudnik

Attachment: 5975_3.patch added

comment:8 Changed 10 years ago by Tobiasz Cudnik

Status: review_failedreview

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

Status: reviewreview_passed

comment:10 Changed 10 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5867].

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