Merge pull request #16058 from ckeditor/ck/6025

Other (core): Clarified the description of keystrokes that execute various buttons (Space, Enter) in the accessibility help dialog.
ckeditor · Mar 27, 2024 · 2b90813 · 2b90813
2 parents 11172e2 + f6a7251
commit 2b90813
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/packages/ckeditor5-core/lang/contexts.json b/packages/ckeditor5-core/lang/contexts.json
@@ -23,5 +23,5 @@
 	"Move focus between form fields (inputs, buttons, etc.)": "Keystroke description for assistive technologies: keystroke for moving between fields.",
 	"Move focus to the toolbar, navigate between toolbars": "Keystroke description for assistive technologies: keystroke for moving focus to the toolbar.",
 	"Navigate through the toolbar": "Keystroke description for assistive technologies: keystroke for navigating through the toolbar.",
-	"Execute the currently focused button": "Keystroke description for assistive technologies: keystroke for executing currently focused button."
+	"Execute the currently focused button. Executing buttons that interact with the editor content moves the focus back to the content.": "Keystroke description for assistive technologies: keystroke for executing currently focused button."
 }
diff --git a/packages/ckeditor5-core/src/accessibility.ts b/packages/ckeditor5-core/src/accessibility.ts
@@ -91,7 +91,8 @@ export default class Accessibility {
 							keystroke: [ [ 'arrowup' ], [ 'arrowright' ], [ 'arrowdown' ], [ 'arrowleft' ] ]
 						},
 						{
-							label: t( 'Execute the currently focused button' ),
+							// eslint-disable-next-line max-len
+							label: t( 'Execute the currently focused button. Executing buttons that interact with the editor content moves the focus back to the content.' ),
 							keystroke: [ [ 'Enter' ], [ 'Space' ] ]
 						}
 					]

diff --git a/packages/ckeditor5-core/tests/accessibility.js b/packages/ckeditor5-core/tests/accessibility.js
@@ -80,7 +80,8 @@ describe( 'Accessibility', () => {
 											[ 'Enter' ],
 											[ 'Space' ]
 										],
-										label: 'Execute the currently focused button'
+										// eslint-disable-next-line max-len
+										label: 'Execute the currently focused button. Executing buttons that interact with the editor content moves the focus back to the content.'
 									}
 								],
 								label: undefined