Opened 12 years ago
Closed 10 years ago
#9638 closed Bug (fixed)
a11yhelp availablity
Reported by: | Garry Yao | Owned by: | Piotrek Koszuliński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.4.3 |
Component: | Accessibility | Version: | 4.0 |
Keywords: | IBM | Cc: | irinauru@…, satya_minnekanti@… |
Description
With preset-basic build profile, the editor is built without the a11yhelp plugin, this causes a issue as the ARIA information still prompts the ALT-0 help info.
Change History (17)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Milestone: | CKEditor 4.0 → CKEditor 4.0.1 |
---|---|
Status: | new → confirmed |
comment:4 Changed 12 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 12 years ago by
Milestone: | CKEditor 4.0.1 |
---|
comment:6 Changed 12 years ago by
Owner: | Garry Yao deleted |
---|---|
Status: | assigned → confirmed |
comment:7 Changed 12 years ago by
Ok, #9541 is closed now. Can we get the preset-basic built with a11yhelp plugin?
comment:8 Changed 11 years ago by
Cc: | irinauru@… added |
---|---|
Keywords: | IBM added |
comment:10 Changed 11 years ago by
Owner: | set to Piotrek Koszuliński |
---|---|
Status: | confirmed → assigned |
comment:11 follow-up: 12 Changed 11 years ago by
Milestone: | → CKEditor 4.4.3 |
---|
Solution for this needs to be based on an even (e.g. editor#ariaEditableLabel, or more specific referring to a11y help), because a11y must be able to mark its presence. The event will return expected label that will be used by wysiwygarea plugin or editable (title is set in both places).
There are, however, two questions which answers I'm not sure:
- What with the .cke_voice_label element and aria-describedby attribute if there's no a11y plugin? Should they be not created at all?
- In wysiwygarea, on IE we prepend
editor.title
to thelang.common.editorHelp
. If a11y help will be missing should this title be still prepended?
comment:12 follow-up: 13 Changed 11 years ago by
Replying to Reinmar:
- What with the .cke_voice_label element and aria-describedby attribute if there's no a11y plugin? Should they be not created at all?
I think so. Accessibility should still be available, even if the a11yhelp plugin is not there.
- In wysiwygarea, on IE we prepend
editor.title
to thelang.common.editorHelp
. If a11y help will be missing should this title be still prepended?
No. Shouldn't this be achieved by the proposed event as well?
comment:13 follow-up: 14 Changed 11 years ago by
Replying to fredck:
Replying to Reinmar:
- What with the .cke_voice_label element and aria-describedby attribute if there's no a11y plugin? Should they be not created at all?
I think so. Accessibility should still be available, even if the a11yhelp plugin is not there.
I don't understand. Maybe my question was not clear. We use .cke_voice_label only for the lang.common.editorHelp
(IE & wysiwygarea is an exception - see next point). If there's no a11yhelp plugin this element will be empty. Should it be created or not in this case? I think that it shouldn't be created.
- In wysiwygarea, on IE we prepend
editor.title
to thelang.common.editorHelp
. If a11y help will be missing should this title be still prepended?No. Shouldn't this be achieved by the proposed event as well?
Event solves adding conditionally lang.common.editorHelp
. Then we've got editor.title
prepended but only on IE and only in wysiwygarea. So I think this logic has to be kept there, in the wysiwygarea plugin. The question was whether editor.title
should be added when there's no a11yhelp plugin, but I understand that no.
comment:14 Changed 11 years ago by
Replying to Reinmar: Yes, I think attributes should not be created/filled if they're supposed to be empty. But really, only during implementation and testing we'll be able to figure this out. It's something to get decided on the spot.
comment:16 Changed 10 years ago by
Status: | review → review_passed |
---|
Seems to be ok, pushed minor changes.
comment:17 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with git:7419ad5844 (merged to master) at dev.
It's a similar issue to #9541. Since we postponed that one, we perhaps should do the same with this one.