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

Focus is not kept inside the multi-save save panel #30943

Closed
carolinan opened this issue Apr 18, 2021 · 1 comment
Closed

Focus is not kept inside the multi-save save panel #30943

carolinan opened this issue Apr 18, 2021 · 1 comment
Labels
[Feature] Saving Related to saving functionality [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended

Comments

@carolinan
Copy link
Contributor

carolinan commented Apr 18, 2021

Description

This refers to the save panel where you can select and save changes to template parts, site blocks, and the page/template itself either in the site editor, block editor or template editor.

When the panel is open and focus is inside the panel, it is possible to leave the save panel without closing it by tabbing backwards, or tabbing forwards from the last item.
The focus is not kept inside the open panel. This is inconsistent with other save panels.

This means that you can navigate around the editor, and once you reach the buttons in the right part of the top bar, (for example, preview or save draft) the save panel is still open and covers the buttons.


-It is unclear to me why there are several different panels used for saving, to avoid this, there should be one, accessible and reusable panel.


Step-by-step reproduction instructions

Open an editor view that uses this panel.
Make a change and open the save panel. Make sure that the focus is in the panel. You may need to click inside of it, or enable
#30873
Use tab to navigate through the panel.
Confirm that when you move backwards all the way past the top, or forwards from the last item, focus moves outside the panel and the panel remains open.

Expected behaviour

Focus needs to be kept inside the panel until the user closes it.

Actual behaviour

Focus is not kept inside the panel.

Screenshots or screen recording (optional)

WordPress information

Device information

  • Device: Desktop
  • Operating system: Windows 10
  • Browser: Chrome Version 89.0.4389.128
@carolinan carolinan added [a11y] Keyboard & Focus [Type] Bug An existing feature does not function as intended [Feature] Saving Related to saving functionality labels Apr 18, 2021
@vdwijngaert
Copy link
Member

I fixed this by adding the withConstrainedTabbing HOC, as is the case with the pre-publish panel in the post editor.

PR: #31136

@priethor priethor added the [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Saving Related to saving functionality [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

4 participants