Opened 6 years ago

Closed 3 years ago

#9086 closed Bug (fixed)

Invalid ARIA property used on paste iframe

Reported by: Damian Owned by: Marek Lewandowski
Priority: Normal Milestone: CKEditor 4.4.8
Component: Accessibility Version: 3.2
Keywords: IBM Cc: Satya Minnekanti, Teresa Monahan

Description

The "Paste Area" IFrame in the Paste dialog utilizes a non-existent WAI-ARIA attribute: "aria-multiple"

<iframe class="cke_pasteframe" ... aria-multiple="true" ... role="region">

Since this IFrame is currently marked up as a region, it should not contain aria-multiple or aria-multiline (which is what probably was originally intended).

This ticket requests simply removing aria-multiple from the region markup.

Change History (7)

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed
Version: 3.2

comment:2 Changed 5 years ago by Mike Gifford

At least in Drupal 8, I don't see the aria-multiple="true"

<iframe class="cke_wysiwyg_frame cke_reset" frameborder="0" style="width: 100%; height: 100%;" aria-describedby="cke_26" title="Rich Text Editor,edit-body-und-0-value" src="" tabindex="0" allowtransparency="true">

Can we assume this has been fixed?

comment:3 Changed 5 years ago by Jakub Ś

@mgifford what damo was talking about was paste dialog (not editor content area) - there is iframe inside it that has this attribute. In fact all three paste dialogs have this attribute so this is still an issue in CKEditor 4.x

comment:4 Changed 5 years ago by Mike Gifford

Thanks for the additional context. Really appreciate it.

comment:5 Changed 3 years ago by Inga

any update on this issue? It is still coming up as a high severity violation in our AVT scans.

Also the rules became more strict now, and role="region" is not acceptable for iframe element - this is coming up as a violation as well.

iframe element must have either no role, or role must be either application, document, or img. Please see the detailed rules here: http://www.w3.org/TR/2012/CR-html5-20121217/dom.html#sec-implicit-aria-semantics

can this be fixed ASAP? Thanks!

comment:6 Changed 3 years ago by Marek Lewandowski

Owner: set to Marek Lewandowski
Status: confirmedreview

Pushed to t/9086.

comment:7 Changed 3 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.4.8
Resolution: fixed
Status: reviewclosed

Fixed on master with git:c5586fe.

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