Skip to content

Fix collapse button overlapping details panel content - #69240

Merged
choo121600 merged 1 commit into
apache:mainfrom
astronomer:fix-details-panel-collapse-button-overlap
Jul 2, 2026
Merged

Fix collapse button overlapping details panel content#69240
choo121600 merged 1 commit into
apache:mainfrom
astronomer:fix-details-panel-collapse-button-overlap

Conversation

@pierrejeambrun

@pierrejeambrun pierrejeambrun commented Jul 2, 2026

Copy link
Copy Markdown
Member

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). This moves the handle onto the resize separator, where a collapse control belongs, and insets the details content so the handle stays clear of it. Collapse behavior is unchanged.

closes: #68754

Before / After

  • Before:
Screenshot 2026-07-02 at 13 23 59
  • After:
Screenshot 2026-07-02 at 13 22 49 Screenshot 2026-07-02 at 13 22 21 Screenshot 2026-07-02 at 13 22 30
  • After (RTL):
Screenshot 2026-07-02 at 13 21 56 Screenshot 2026-07-02 at 13 22 06
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

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Jul 2, 2026
@pierrejeambrun
pierrejeambrun marked this pull request as ready for review July 2, 2026 09:47
@pierrejeambrun
pierrejeambrun force-pushed the fix-details-panel-collapse-button-overlap branch from 5499707 to ba7ce89 Compare July 2, 2026 09:58
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: apache#68754

@choo121600 choo121600 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.

looks good to me 👍

@choo121600
choo121600 merged commit 44b7049 into apache:main Jul 2, 2026
85 checks passed
@vatsrahul1001 vatsrahul1001 added the backport-to-v3-3-test Backport to v3-3-test label Jul 30, 2026
vatsrahul1001 pushed a commit that referenced this pull request Jul 30, 2026
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)
pierrejeambrun added a commit that referenced this pull request Jul 30, 2026
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)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
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. backport-to-v3-3-test Backport to v3-3-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

'Task State Store' or 'Xcom' key is getting overlapped with expand/collapse details panel

3 participants