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

Cannot change list style with keyboard #11041

Closed
LukaszGudel opened this issue Dec 23, 2021 · 4 comments · Fixed by #12202
Closed

Cannot change list style with keyboard #11041

LukaszGudel opened this issue Dec 23, 2021 · 4 comments · Fixed by #12202
Assignees
Labels
domain:accessibility This issue reports an accessibility problem. package:list squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.

Comments

@LukaszGudel
Copy link
Contributor

📝 Provide detailed reproduction steps (if any)

  1. Go to https://ckeditor5.github.io/docs/nightly/ckeditor5/latest/features/lists/lists.html#demo-2
  2. Focus the editor and press Alt+F10.
  3. Use keyboard arrows to navigate to list styles dropdown.
  4. Open dropdown with space or enter key.
  5. Try to select a different list style with arrows.

✔️ Expected result

You are able to use arrows to choose a different style.

❌ Actual result

After opening the dropdown you are unable to change the style with arrows.

Notes:


If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@LukaszGudel LukaszGudel added type:bug This issue reports a buggy (incorrect) behavior. package:list type:regression This issue reports a bug that was not present in the previous releases. squad:core Issue to be handled by the Core team. domain:accessibility This issue reports an accessibility problem. labels Dec 23, 2021
@Reinmar Reinmar added the release:blocker This issue blocks the upcoming release (is critical). label Dec 23, 2021
@Reinmar
Copy link
Member

Reinmar commented Dec 30, 2021

Today:

actions: {
// Navigate #children backwards using the <kbd>Shift</kbd> + <kbd>Tab</kbd> keystroke.
focusPrevious: 'shift + tab',
// Navigate #children forwards using the <kbd>Tab</kbd> key.
focusNext: 'tab'
}

Previously, we used toolbar view which supports arrows:

actions: {
// Navigate toolbar items backwards using the arrow[left,up] keys.
focusPrevious: [ isRtl ? 'arrowright' : 'arrowleft', 'arrowup' ],
// Navigate toolbar items forwards using the arrow[right,down] keys.
focusNext: [ isRtl ? 'arrowleft' : 'arrowright', 'arrowdown' ]
}

This means that the behavior changed – currently you need to use the Tab key, previously arrows worked. I think that arrows should still work, but it requires a deeper analysis of what should Tab do and what arrows in this entire panel.

@Reinmar Reinmar removed the release:blocker This issue blocks the upcoming release (is critical). label Dec 30, 2021
@bbenjamin
Copy link

Tabs don't work for me, that results in focus leaving the toolbar entirely. I also think that's a reasonable expectation for tab and the arrow support should come back. Aside from it being a good pattern to expect, it's also what how highlighter currently works and it's probably best if there's consistency even if there are potential future changes to keyboard navigation https://ckeditor.com/docs/ckeditor5/latest/examples/builds-custom/full-featured-editor.html

@mlewand mlewand added squad:features Issue to be handled by the Features team. and removed squad:core Issue to be handled by the Core team. labels May 27, 2022
@mlewand
Copy link
Contributor

mlewand commented May 27, 2022

Arrows should definitely be handled - that's the first thing that should work for grid-like components (like list styles). Regarding the tab we need to investigate it further to see what would be the most optimal solution.

@wimleers
Copy link

FYI: tracking this at https://www.drupal.org/project/drupal/issues/3283800 😊

oleq added a commit that referenced this issue Aug 12, 2022
…r-grid-components-discovery-poc

Feature (ui): Introduced the `addKeyboardHandlingForGrid()` helper to handle grid keyboard navigation. Closes #11851.

Fix (font): Fixed focus order for color grid in font color and background drop-downs. Closes #11841.

Fix (list): Unified focus handling and keyboard navigation in list properties drop-downs. Closes #11041.
@mlewand mlewand added this to the iteration 56 milestone Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:accessibility This issue reports an accessibility problem. package:list squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants