Skip to content

Commit 7c90e7f

Browse files
committed
♿️(frontend) fix subdoc opening and emoji pick focus
ensures subdoc opens and emoji picker focus on input Signed-off-by: Cyril <c.gromoff@gmail.com>
1 parent f779d2a commit 7c90e7f

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ and this project adheres to
2121
- 🌐(backend) internationalize demo #1644
2222
- ♿(frontend) improve accessibility:
2323
- ♿️Improve keyboard accessibility for the document tree #1681
24-
- ♿️(frontend) fix subdoc opening and emoji pick focus
24+
- ♿️(frontend) fix subdoc opening and emoji pick focus #1745
2525

2626
### Fixed
2727

src/frontend/apps/impress/src/features/docs/doc-editor/components/EmojiPicker.tsx

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,20 +20,19 @@ export const EmojiPicker = ({
2020
}: EmojiPickerProps) => {
2121
const { i18n } = useTranslation();
2222

23-
// Close picker with Escape key for keyboard users
23+
// Handle Escape key to close the picker
2424
useEffect(() => {
25-
const handleKeyDown = (event: KeyboardEvent) => {
26-
if (event.key === 'Escape') {
27-
event.stopPropagation();
28-
event.preventDefault();
25+
const handleEscape = (e: KeyboardEvent) => {
26+
if (e.key === 'Escape') {
27+
e.preventDefault();
28+
e.stopPropagation();
2929
onClickOutside();
3030
}
3131
};
3232

33-
window.addEventListener('keydown', handleKeyDown, true);
34-
33+
document.addEventListener('keydown', handleEscape, true);
3534
return () => {
36-
window.removeEventListener('keydown', handleKeyDown, true);
35+
document.removeEventListener('keydown', handleEscape, true);
3736
};
3837
}, [onClickOutside]);
3938

0 commit comments

Comments
 (0)