Skip to content

[v3-3-test] Fix collapse button overlapping details panel content (#69240) - #70751

Merged
pierrejeambrun merged 1 commit into
v3-3-testfrom
backport-44b7049-v3-3-test
Jul 30, 2026
Merged

[v3-3-test] Fix collapse button overlapping details panel content (#69240)#70751
pierrejeambrun merged 1 commit into
v3-3-testfrom
backport-44b7049-v3-3-test

Conversation

@vatsrahul1001

@vatsrahul1001 vatsrahul1001 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Backport of #69240 to v3-3-test for the Airflow 3.3.1 patch release.

DetailsLayout.tsx had diverged on v3-3-test, so this block conflicted. Resolved by taking #69240's version — move the collapse IconButton into the resize-handle <Box>, restyle it (borderRadius="full", insetInlineStart="50%" + transform centering), and add paddingInlineStart={4} to the details panel. The UI format/lint prek hook normalized indentation; tags balance and the warning button / DAGWarningsModal remain siblings.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines — cherry-pick backport with a manually-resolved structural conflict as described above.

The details-panel collapse handle was rendered inside the details panel — which
clips overflow — and anchored to the panel's inner edge, so it sat over the
panel content and covered the leftmost column (the XCom / Task State Store key
reported in the issue). The resize separator is where a collapse handle belongs,
clear of either panel's content.

closes: #68754
(cherry picked from commit 44b7049)
@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.3.1 milestone Jul 30, 2026
@vatsrahul1001 vatsrahul1001 added the type:bug-fix Changelog: Bug Fixes label Jul 30, 2026
@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Jul 30, 2026

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tested and working as expected

@pierrejeambrun
pierrejeambrun merged commit 34d9c9c into v3-3-test Jul 30, 2026
144 of 145 checks passed
@pierrejeambrun
pierrejeambrun deleted the backport-44b7049-v3-3-test branch July 30, 2026 14:46
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. type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants