Skip to content

Commit 5199cee

Browse files
fix: remove extra comments & add description for handleKeyDown
1 parent 8eb8ab0 commit 5199cee

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

src/courseware/course/sidebar/common/hooks/useSidebarFocusAndKeyboard.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,13 @@ export const useSidebarFocusAndKeyboard = (sidebarId, triggerButtonSelector = '.
4545
focusSidebarTriggerBtn();
4646
}, [toggleSidebar, focusSidebarTriggerBtn]);
4747

48+
/**
49+
* Handles Tab key navigation when focus is on the standard sidebar close button.
50+
* Implements the logic for moving focus out of the sidebar to specific elements
51+
* on the main page in a predefined sequence, or back to the trigger button on Shift+Tab.
52+
*
53+
* @param {KeyboardEvent} event - The keyboard event object.
54+
*/
4855
const handleKeyDown = useCallback((event) => {
4956
if (event.key !== 'Tab') {
5057
return;

src/courseware/course/sidebar/common/hooks/useSidebarFocusAndKeyboard.test.jsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
// hooks/useSidebarFocusAndKeyboard.test.js (або .jsx)
21
import { renderHook, act } from '@testing-library/react';
32
import SidebarContext from '../../SidebarContext';
43
import { useSidebarFocusAndKeyboard } from './useSidebarFocusAndKeyboard';
@@ -165,7 +164,6 @@ describe('useSidebarFocusAndKeyboard', () => {
165164
});
166165

167166
expect(mockEvent.preventDefault).toHaveBeenCalledTimes(1);
168-
// Перевіряємо фокус тригера
169167
act(() => { jest.runAllTimers(); });
170168
expect(triggerButtonMock.focus).toHaveBeenCalledTimes(1);
171169
expect(tryFocusAndPreventDefault).not.toHaveBeenCalled();

0 commit comments

Comments
 (0)