Opened 13 years ago
Closed 10 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 13 years ago by
Status: | new → confirmed |
---|---|
Version: | → 3.2 |
comment:2 Changed 12 years ago by
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 12 years ago by
@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:5 Changed 10 years ago by
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 10 years ago by
Owner: | set to Marek Lewandowski |
---|---|
Status: | confirmed → review |
Pushed to t/9086.
comment:7 Changed 10 years ago by
Milestone: | → CKEditor 4.4.8 |
---|---|
Resolution: | → fixed |
Status: | review → closed |
Fixed on master with git:c5586fe.
http://www.w3.org/TR/wai-aria/states_and_properties#aria-multiline