-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 save button when entities save states panel is opened #30873
Conversation
Size Change: +71 B (0%) Total Size: 1.46 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested in both editors and this works well. Thanks for the fix 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested in the site editor, post editor and template editing mode in Chrome on Windows 10 with and without NVDA screen reader.
The focus works and it is possible to save now.
@talldan, any reason for Backport label? This should already be in core based on the Gutenberg version. |
Not sure, it definitely doesn't need to be backported. Seems like I added the label accidentally. |
Fixes #29803
Description
Moves focus into the entities save states panel when it is opened via "Save" / "Publish" button. This change applies to both
post-edit
andsite-edit
. Although this panel is only shown in the Post Editor when you make changes to other entities. (For example you edit a template part)How has this been tested?
Site Editor
Post Editor
Screenshots
Screen.Recording.2021-04-15.at.20.23.53.mov
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist:
*.native.js
files for terms that need renaming or removal).