Skip to content

Conversation

dariiing
Copy link
Contributor

@dariiing dariiing commented Oct 1, 2024

This PR fixes a keyboard trap found inside the Form description. I've added the Esc option in order to remove the focus from the text field (this option is also used for other CodeMirror usages), fixing #1638.

@dariiing dariiing added Accessibility Make software available to everyone Form Related to the Form app Ready to Review This PR is ready to be checked by another team member HAN scan 08-24 Accessibility scan Keyboard navigation labels Oct 1, 2024
@dariiing dariiing added this to the Winter release (2024.12.0) milestone Oct 1, 2024
@dariiing dariiing self-assigned this Oct 1, 2024
Copy link
Contributor

@Chouhartem Chouhartem left a comment

Choose a reason for hiding this comment

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

LGTM

@yflory yflory removed the Ready to Review This PR is ready to be checked by another team member label Dec 3, 2024
@yflory yflory merged commit 922b64b into staging Dec 3, 2024
@mathilde-cryptpad mathilde-cryptpad deleted the form-description-keyboard-fixes branch December 18, 2024 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Make software available to everyone Form Related to the Form app HAN scan 08-24 Accessibility scan Keyboard navigation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants