-
-
Notifications
You must be signed in to change notification settings - Fork 771
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
HtmlSlateWidget to proper identify the field to screen readers #6772
base: 17.x.x
Are you sure you want to change the base?
Conversation
@Wagner3UB some tests are failing, probably because you have to update snapshots.. |
@Wagner3UB there are some test still failing.. check them |
@davisagli Can you help me here? I don’t think the problem is related to my changes. Cypress is returning an error in the file field.js when it tries to create a new text block from the ‘most used’ ones, but the text block isn’t there. In other words, the text block is not one of the most used ones.
|
@Wagner3UB I had something like this when running tests locally and found out that I was not running with the correct fixture. Maybe try running with coresandbox fixtures specifically? |
Correction applied to ensure proper screen reader identification of text fields created through the HtmlSlateWidget.
PR volto 18 #6803