fix: OnNavigationChange callback (#3542)

Author: georgylobko

src/app-layout/__tests__/app-layout-navigation.test.tsx

@@ -42,4 +42,22 @@ describeEachAppLayout({ themes: ['refresh-toolbar'], sizes: ['desktop', 'mobile'
     expect(wrapper.findNavigation()).toBeTruthy();
     expect(wrapper.findNavigationToggle()).toBeFalsy();
   });
+
+  (size === 'desktop' ? test : test.skip)('should call onNavigationToggle on open/close navigation', () => {
+    const mockOnNavigationChange = jest.fn();
+    const { wrapper } = renderComponent(
+      <AppLayout navigation={<>Mock Navigation</>} onNavigationChange={mockOnNavigationChange} content={<>Content</>} />
+    );
+
+    expect(wrapper.findOpenNavigationPanel()).toBeTruthy();
+    wrapper.findNavigationToggle().click();
+    expect(mockOnNavigationChange).toHaveBeenCalledTimes(1);
+    expect(mockOnNavigationChange).toHaveBeenCalledWith(expect.objectContaining({ detail: { open: false } }));
+    expect(wrapper.findOpenNavigationPanel()).toBeFalsy();
+
+    wrapper.findNavigationToggle().click();
+    expect(mockOnNavigationChange).toHaveBeenCalledTimes(2);
+    expect(mockOnNavigationChange).toHaveBeenCalledWith(expect.objectContaining({ detail: { open: true } }));
+    expect(wrapper.findOpenNavigationPanel()).toBeTruthy();
+  });
 });

src/app-layout/visual-refresh-toolbar/toolbar/index.tsx

@@ -171,7 +171,7 @@ export function AppLayoutToolbarImplementation({
                 if (setExpandedDrawerId) {
                   setExpandedDrawerId(null);
                 }
-                if (navigationOpen) {
+                if (navigationOpen && expandedDrawerId) {
                   return;
                 }
                 onNavigationToggle?.(!navigationOpen);