Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accessibility: Editor Control Type Incorrect #4052

Closed
lumills opened this issue May 13, 2020 · 10 comments · Fixed by #5146
Closed

Accessibility: Editor Control Type Incorrect #4052

lumills opened this issue May 13, 2020 · 10 comments · Fixed by #5146
Assignees
Labels
accessibility Issue related to accessibility. status:confirmed An issue confirmed by the development team. support:2 An issue reported by a commercially licensed client. type:bug A bug.
Milestone

Comments

@lumills
Copy link

lumills commented May 13, 2020

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Open a ckeditor 4 instance
  2. Turn screen reader on (NVDA works)
  3. Navigate into the Editor itself
  4. Control type should be announced

Expected result

Control type should be meaningful to the user, such as 'Editor'

Actual result

Control type is not informational (I believe it is announced as 'group')

Other details

  • Browser: Firefox
  • CKEditor version: 4
@lumills lumills added the type:bug A bug. label May 13, 2020
@lslowikowska lslowikowska added the support:2 An issue reported by a commercially licensed client. label May 14, 2020
@f1ames
Copy link
Contributor

f1ames commented May 14, 2020

Hi @lumills,

does it only concerns NVDA or any other screen reader too?

@f1ames f1ames added accessibility Issue related to accessibility. status:pending labels May 14, 2020
@lumills
Copy link
Author

lumills commented May 14, 2020

Hi @f1ames ,

Bug and repro that I'm seeing appears to be NVDA specifically

@Comandeer Comandeer self-assigned this May 21, 2020
@Comandeer
Copy link
Member

I can confirm that editor is presented in unhelpful way. It seems to be connected with a convoluted way in which we label the editor – see #2445, #2499 and #1904.

@Comandeer Comandeer added the status:confirmed An issue confirmed by the development team. label May 21, 2020
@Comandeer Comandeer removed their assignment May 21, 2020
@lumills
Copy link
Author

lumills commented Sep 2, 2020

Hey @f1ames and @Comandeer just wanted to check if there is any sort of eta here / confirmation that it is in the pipeline? Thanks in advance

@f1ames
Copy link
Contributor

f1ames commented Sep 3, 2020

Hello @lumills, unfortunately we have no ETA for this one. We have briefly analyzed this issue and related ones (see #4052 (comment)). This requires a solid rework on our side regarding how editor markup is created and so it's really hard to estimate how and when we can have a working solution.

Still, we are keeping an eye on it as accessibility is very important for us so there is a chance there will be a fix in a foreseeable future.

@naksenzh
Copy link

naksenzh commented Oct 9, 2020

Hello @f1ames, we are tracking this issue as a blocker for our accessibility bugs. We would appreciate a lot if you could share the updates, please.

@f1ames
Copy link
Contributor

f1ames commented Oct 9, 2020

Hello @naksenzh, as mentioned in #4052 (comment), we have no clear ETA for this issue since it requires a significant amount of work in the editor itself. Any visible progress we will make on this issue (whatever positive or negative) will be posted here so you can track the status.

@naksenzh
Copy link

naksenzh commented Jan 18, 2021

Hi @f1ames any update or progress on this by chance ?

@f1ames
Copy link
Contributor

f1ames commented Jan 19, 2021

Hello @naksenzh, unfortunately due to other priorities we were unable to push this issue forward for now.

@CKEditorBot
Copy link
Collaborator

Closed in #5146

@jacekbogdanski jacekbogdanski removed this from the 4.18.1 milestone May 19, 2022
@jacekbogdanski jacekbogdanski added this to the 4.19.0 milestone May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issue related to accessibility. status:confirmed An issue confirmed by the development team. support:2 An issue reported by a commercially licensed client. type:bug A bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants