SAP / ui5-webcomponents

UI5 Web Components - the enterprise-flavored sugar on top of native APIs! Build SAP Fiori user interfaces with the technology of your choice.
https://sap.github.io/ui5-webcomponents/
Apache License 2.0
1.51k stars 262 forks source link

[SF_ACC][ui5-dialog]: CNTL + Arrow keys not working for draggable dialog #8190

Closed GongRichard closed 6 months ago

GongRichard commented 7 months ago

Bug Description

CNTL + Arrow keys are not working for draggable dialog to move the dialog. Where is it documented for how to resize and move dialogs using the keyboard? We need to tell the customer.

Affected Component

No response

Expected Behaviour

CNTL + Arrow key should be working for draggable dialog to move the dialog. You can ui5 page https://ui5.sap.com/#/entity/sap.m.Dialog/sample/sap.m.sample.Dialog, it works.

Isolated Example

No response

Steps to Reproduce

  1. open page https://sap.github.io/ui5-webcomponents/playground/?path=/story/main-dialog--draggable-and-resizable
  2. Open the dialog
  3. When the dialog is opened, the focus is on the dialog title bar by default. Use CNTL+arrow keys and can't move the dialog.

Log Output, Stack Trace or Screenshots

No response

Priority

Medium

UI5 Web Components Version

1.21.x

Browser

Chrome, Edge, Firefox, Safari

Operating System

IOS

Additional Context

No response

Organization

SAP SuccessFactors

Declaration

petyabegovska commented 7 months ago

Hi @GongRichard,

When the focus is on the dialog title, you should use the [Arrow keys] to move the dialog:

I will send you the Keyboard specification in Microsoft Teams.

Best regards, Petya

petyabegovska commented 7 months ago

Follow up discussion will be triggered how we provide end user documentation similar to SAPUI5: https://help.sap.com/docs/SAPUI5/bc5a64aac808463baa95b4230f221716/144d377d68784689a9b21565dfa14065.html?version=1.96

s-todorova commented 6 months ago

Hi @GongRichard,

With PR#8355, the keyboard navigation for Dialog component is now properly documented.

Best regards, Siyana