Skip to content

[v3-2-test] fix(ui): prevent duplicate nav sidebar when iframe navigates away from auth pages (#63873)#64854

Merged
bbovenzi merged 1 commit intov3-2-testfrom
backport-545baf4-v3-2-test
Apr 7, 2026
Merged

[v3-2-test] fix(ui): prevent duplicate nav sidebar when iframe navigates away from auth pages (#63873)#64854
bbovenzi merged 1 commit intov3-2-testfrom
backport-545baf4-v3-2-test

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions bot commented Apr 7, 2026

When a user opens a Security panel (e.g. Users) and clicks the Airflow
logo inside the embedded FAB iframe, the iframe navigates to '/' which
causes the React SPA to render inside the iframe, including its own Nav
sidebar. This produces a duplicate navigation bar alongside the outer
app's sidebar.

Fix: before calling navigate('/'), set iframe.src = 'about:blank' to
immediately clear the iframe content so the inner React app has no
chance to render its Nav. A isRedirecting ref guards against the extra
onLoad event that setting src to about:blank would otherwise trigger.

Closes #63805
(cherry picked from commit 545baf4)

Co-authored-by: nagasrisai 59650078+nagasrisai@users.noreply.github.com

…tes away from auth pages (#63873)

When a user opens a Security panel (e.g. Users) and clicks the Airflow
logo inside the embedded FAB iframe, the iframe navigates to '/' which
causes the React SPA to render inside the iframe, including its own Nav
sidebar. This produces a duplicate navigation bar alongside the outer
app's sidebar.

Fix: before calling navigate('/'), set iframe.src = 'about:blank' to
immediately clear the iframe content so the inner React app has no
chance to render its Nav. A isRedirecting ref guards against the extra
onLoad event that setting src to about:blank would otherwise trigger.

Closes #63805
(cherry picked from commit 545baf4)

Co-authored-by: nagasrisai <59650078+nagasrisai@users.noreply.github.com>
@bbovenzi bbovenzi marked this pull request as ready for review April 7, 2026 17:52
@bbovenzi bbovenzi merged commit 11db467 into v3-2-test Apr 7, 2026
77 checks passed
@bbovenzi bbovenzi deleted the backport-545baf4-v3-2-test branch April 7, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants