From e99e5225730a832f29fc45790786c864d80f5c04 Mon Sep 17 00:00:00 2001 From: Jack Kingsman Date: Sun, 8 Mar 2026 22:17:32 -0700 Subject: [PATCH] Fix clipping on integration add drop down --- app/path_utils.py | 6 +++++- frontend/src/components/SettingsModal.tsx | 2 +- frontend/src/test/settingsModal.test.tsx | 11 +++++++++++ 3 files changed, 17 insertions(+), 2 deletions(-) diff --git a/app/path_utils.py b/app/path_utils.py index 1b9c294..eb19c89 100644 --- a/app/path_utils.py +++ b/app/path_utils.py @@ -194,7 +194,11 @@ def normalize_contact_route( expected_bytes = normalized_len * bytes_per_hop if actual_bytes > expected_bytes >= 0: normalized_path = normalized_path[: expected_bytes * 2] - elif actual_bytes < expected_bytes and bytes_per_hop > 0 and actual_bytes % bytes_per_hop == 0: + elif ( + actual_bytes < expected_bytes + and bytes_per_hop > 0 + and actual_bytes % bytes_per_hop == 0 + ): normalized_len = actual_bytes // bytes_per_hop return normalized_path, normalized_len, normalized_mode diff --git a/frontend/src/components/SettingsModal.tsx b/frontend/src/components/SettingsModal.tsx index f056821..99feb44 100644 --- a/frontend/src/components/SettingsModal.tsx +++ b/frontend/src/components/SettingsModal.tsx @@ -123,7 +123,7 @@ export function SettingsModal(props: SettingsModalProps) { const shouldRenderSection = (section: SettingsSection) => !externalDesktopSidebarMode || desktopSection === section; - const sectionWrapperClass = 'overflow-hidden'; + const sectionWrapperClass = ''; const sectionContentClass = externalDesktopSidebarMode ? 'mx-auto w-full max-w-[800px] space-y-4 p-4' diff --git a/frontend/src/test/settingsModal.test.tsx b/frontend/src/test/settingsModal.test.tsx index 18206d0..1f2ae0e 100644 --- a/frontend/src/test/settingsModal.test.tsx +++ b/frontend/src/test/settingsModal.test.tsx @@ -231,6 +231,17 @@ describe('SettingsModal', () => { expect(screen.queryByLabelText('Preset')).not.toBeInTheDocument(); }); + it('does not clip the fanout add-integration menu in external desktop mode', () => { + renderModal({ + externalSidebarNav: true, + desktopSection: 'fanout', + }); + + const addIntegrationButton = screen.getByRole('button', { name: 'Add Integration' }); + const wrapperSection = addIntegrationButton.closest('section'); + expect(wrapperSection).not.toHaveClass('overflow-hidden'); + }); + it('applies the centered 800px column layout to non-fanout settings content', () => { renderModal({ externalSidebarNav: true,