Ticket #7745 (closed Bug: fixed)
Instructions on how to access help dialog should be placed in an aria-describedby attribute
| Reported by: | satya | Owned by: | fredck |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.4 |
| Component: | Accessibility | Version: | |
| Keywords: | IBM VendorFix | Cc: | damo, jamescun, tmonahan |
Description
This needs to be changed because when we bring up the list of frames on a page using JAWS, it shows CK Editor but also the extra help statement that we kept in the title attribute.This will confuse the users & we don't need this when identifying frames, we use that statement only for JAWS users on how to find help dialog.
Best way to fix this is to keep the Help Instructions in an aria-describedby attribute instead of on the Title attribute for the content area.
Attachments
Change History
comment:1 Changed 2 years ago by fredck
- Status changed from new to confirmed
- Component changed from General to Accessibility
comment:2 follow-up: ↓ 3 Changed 2 years ago by garry.yao
Exactly as Fred explained, we cannot get aria label work for wysiwyg mode (on iframe) unless FS could give us some hints of it, this is hold for all browsers up till now.
comment:3 in reply to: ↑ 2 Changed 2 years ago by damo
Replying to fredck:
We need to check whether changing it for Firefox is ok, making it a special case, still leaving the title thing there for other browsers.
We're not really looking for a browser specific fix in this case.
Replying to garry.yao:
Exactly as Fred explained, we cannot get aria label work for wysiwyg mode (on iframe) unless FS could give us some hints of it, this is hold for all browsers up till now.
Ok, I'll try to investigate how this is supposed to work with FS.
comment:5 Changed 23 months ago by damo
This has been addressed in JAWS 12.0.1167.
Frames should be able to have the aria-describedby attribute like any other component. JAWS now recognizes descriptions applied to Frames.
comment:6 Changed 21 months ago by fredck
- Status changed from pending to confirmed
We need to check it with the latest JAWS at this point.
comment:7 Changed 21 months ago by garry.yao
- Keywords VendorFix added
By applying the patch, at least IE will not announce anymore the editor instructions upon focused, so this hack has to be remained.
comment:8 follow-up: ↓ 10 Changed 18 months ago by damo
The patch sets an aria-label on the iframe, but it should be setting aria-describedby instead.
We should still keep the title attribute on the iframe, then we won't need an aria-label.
comment:10 in reply to: ↑ 8 ; follow-up: ↓ 11 Changed 15 months ago by garry.yao
Replying to damo:
The patch sets an aria-label on the iframe, but it should be setting aria-describedby instead.
We should still keep the title attribute on the iframe, then we won't need an aria-label.
It's not working...please check the tc attached.
comment:11 in reply to: ↑ 10 Changed 15 months ago by damo
Replying to garry.yao:
It's not working...please check the tc attached.
I tried this with FF 10 and JAWS 13, when focusing on the IFrame it reads the description.
There is still one problem with the tc, it does not set a name for the IFrame. I propose that you put a title on the IFrame, as you have been doing up till now. So, then the aria-describedby attribute only contributes the "Press Alt 0 for help" text.
E.g.
<iframe title="Rich Text Editor, editor1" aria-describedby="id1" ...> <span id="id1">Press Alt 0 for help</span>
Also, I have used Inspect32 to test that the description is properly being conveyed by the browser in your tc, see attached screen shot.
Changed 15 months ago by damo
- Attachment inspect32_result.png added
Example Inspect32 result for tc
comment:12 Changed 15 months ago by garry.yao
The problem is not FF, could you check it in IE with JAWS 13?
comment:13 Changed 14 months ago by garry.yao
- Owner set to garry.yao
- Status changed from confirmed to review
The patch follows the standard way of describing the editable frame in ARIA, while still provide the fallback for JAWS to work properly with IE, so the JAWS announcement would be:
- IE : Rich text editor, editor1, press alt 0 for help - edit
- FF: Rich text editor - edit, editor1, press alt 0 for help
Changed 14 months ago by garry.yao
- Attachment 7745_2.patch added
path compensate description with aria live region
comment:14 Changed 14 months ago by garry.yao
Another new patch that takes the ARIA live region approach to announce the frame description, at least with JAWS13, the result is browser independent:
Rich text editor (label) - edit (role) - blah, blah (text line) - press alt 0 for help (description).
comment:16 Changed 14 months ago by fredck
- Status changed from review to assigned
- Milestone CKEditor 3.6.3 deleted
I'm attaching a new patch for it, which makes the minimum changes to satisfy the ticket request.
The patch solves the problem with the frames list (INS+F9).
The following are result we're having with the browsers.
On IE 9:
Before patch: "Rich text editor, editor1, press ALT+0 for help, edit, {read contents}, type in text" (Good)
After patch: "Rich text editor, editor1, press ALT+0 for help, edit, {read contents}, type in text" (Good)
So, no changes at all.
On FF 11:
Before Patch: "Rich text editor, editor1, press ALT+0 for help, frame, edit, {read contents}, type in text"
After Patch: "Rich text editor, editor1, frame, press ALT+0 for help, edit, {read contents}, press ALT+0 for help, type in text"
Although we have the INS+F9 issue solved, you can see that we have problems with FF, which announces the description twice.
Considering that the patch brings the above issue, we should hold on with this patch until well be able to have Firefox and JAWS working well with it.
I think the focus behavior is more important than the Frames List issue and, sincerely speaking, I would instead try to find a way to not list the editor in the frames list at all, because it is senseless.
comment:17 Changed 13 months ago by damo
The problem with JAWS announcing descriptions twice is a known JAWS defect, being addressed.
I think the focus behavior is more important than the Frames List issue and, sincerely speaking, I would instead try to find a way to not list the editor in the frames list at all, because it is senseless.
We're having some internal discussion about this one. There is some agreement about hiding the frame semantics for the editor.
comment:18 Changed 11 months ago by damo
Considering that JAWS will fix the issue of reading the description twice, we should implement the current fix. It works in FF and IE.
comment:19 Changed 11 months ago by fredck
- Owner changed from garry.yao to fredck
- Status changed from assigned to review
comment:20 Changed 11 months ago by garry.yao
- Status changed from review to review_passed
- Milestone set to CKEditor 3.6.4
comment:21 Changed 11 months ago by fredck
- Status changed from review_passed to closed
- Resolution set to fixed
Fixed with [7499].

If I recall it well, we have put the description in the title because of browser compatibility issues. Having it there enlarged the accessibility support for the announcement instructions to more browsers.
We need to check whether changing it for Firefox is ok, making it a special case, still leaving the title thing there for other browsers.