Skip to content

feat: add backdrop fade-in/out transition to master-detail-layout#11414

Merged
web-padawan merged 2 commits intomainfrom
feat/mdl-backdrop-animate
Mar 25, 2026
Merged

feat: add backdrop fade-in/out transition to master-detail-layout#11414
web-padawan merged 2 commits intomainfrom
feat/mdl-backdrop-animate

Conversation

@web-padawan
Copy link
Copy Markdown
Member

Summary

  • Animate the backdrop opacity alongside the detail panel slide when opening/closing the overlay
  • Switch backdrop from display: none/block to opacity: 0/1 + pointer-events: none/auto so the Web Animations API can fade it
  • Use linear easing for a clean fade independent of the detail panel's cubic-bezier curve
  • Backdrop fade only runs for overlay add/remove (not replace — backdrop stays visible)

Test plan

  • yarn test --group master-detail-layout — all tests pass
  • yarn test:snapshots --group master-detail-layout — pass
  • yarn lint:js / yarn lint:css — pass
  • yarn test:base --group master-detail-layout — visual tests (requires Docker)
  • Manual: open/close overlay detail, verify backdrop fades smoothly
  • Manual: click backdrop during fade-in, verify interruption works

🤖 Generated with Claude Code

Animate the backdrop opacity alongside the detail panel slide when
opening/closing the overlay. Previously the backdrop appeared and
disappeared instantly (display: none/block).

Switch backdrop visibility from display toggling to opacity + pointer-
events so the Web Animations API can fade it. Use linear easing for a
clean fade independent of the detail panel's cubic-bezier curve.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@web-padawan web-padawan requested review from jouni and vursen March 25, 2026 11:47
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

@web-padawan web-padawan merged commit 2a8bc15 into main Mar 25, 2026
9 checks passed
@web-padawan web-padawan deleted the feat/mdl-backdrop-animate branch March 25, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants