-
-
Notifications
You must be signed in to change notification settings - Fork 81
Fix: prevent arrow key nav conflict inside textareas #1445
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
base: main
Are you sure you want to change the base?
Fix: prevent arrow key nav conflict inside textareas #1445
Conversation
👷 Deploy request for cyf-curriculum pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-piscine pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-launch pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-common pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-itd pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-sdc pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-tracks pending review.Visit the deploys page to approve it
|
✅ Deploy Preview for cyf-programming ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for picking this up! I tried this in the deploy preview (https://deploy-preview-1445--cyf-programming.netlify.app/data-groups/sprints/3/prep/#implementing-a-character-limit) and the left/right arrows still seem to have the buggy behaviour. Please could you take a look? I'd recommend running the site locally, and debugging interactively :)
What does this change?
This PR fixes a bug where pressing the left or right arrow keys inside a textarea would trigger navigation to a different prep section instead of moving the cursor within the textarea.
The fix ensures that arrow key navigation handlers ignore key events when the focused element is an input or textarea.
Common Content?
Common Theme?
Issue number: #1072
Org Content?
A function fixed
Module | Sprint | Page Type | Block Type
Checklist
Who needs to know about this?