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

feat: dashboard keyboard interactions #7811

Merged
merged 7 commits into from
Sep 18, 2024

Conversation

tomivirkki
Copy link
Member

Description

Dashboard keyboard interactions as described here.
Widget/section move-mode and resize-mode are excluded from this PR and will be added separately.

Part of https://github.com/orgs/vaadin/projects/70/views/1?pane=issue&itemId=79678587

Type of change

Feature

Copy link
Contributor

@yuriy-fix yuriy-fix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an issue in Safari (Version 17.6): the focus gets lost when trying to Tab to the next widget or after removing one of the widgets.

safari-bug
Steps to reproduce:

  1. Select the first widget with a click.
  2. Press Esc.
  3. Press Tab.

Expected behavior: The next widget is focused.
Actual behavior: The focus is lost.

packages/dashboard/src/widget-resize-controller.js Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Sep 18, 2024

@tomivirkki
Copy link
Member Author

There is an issue in Safari (Version 17.6): the focus gets lost when trying to Tab to the next widget or after removing one of the widgets.

I can't reproduce it, running the same Safari version:

Kapture.2024-09-18.at.16.31.40.mp4

@tomivirkki tomivirkki merged commit be84ceb into feat/dashboard Sep 18, 2024
9 checks passed
@tomivirkki tomivirkki deleted the feat/keyboard-interactions branch September 18, 2024 13:58
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.6.0.beta1 and is also targeting the upcoming stable 24.6.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants