Skip to content

fix: tater sprite z-index regression from mobile compat PR#308

Merged
backnotprop merged 2 commits intomainfrom
fix/ui-hierarchy
Mar 16, 2026
Merged

fix: tater sprite z-index regression from mobile compat PR#308
backnotprop merged 2 commits intomainfrom
fix/ui-hierarchy

Conversation

@backnotprop
Copy link
Copy Markdown
Owner

Summary

  • Tater sprite (running animation) was appearing in front of the plan document instead of behind it
  • Root cause: PR feat: add mobile compatibility #260 (mobile compatibility) added relative z-0 to the content wrapper, creating a new stacking context that trapped the plan at z-0 while the sprite at z-40 (rendered outside the wrapper) floated above everything
  • Fix: move TaterSpriteRunning inside the content wrapper and adjust z-indexes so sprite (z-5) layers above sidebars but below the plan document (z-10)

Test plan

  • Enable tater mode in settings
  • Verify sprite runs behind the plan document card, not in front of it
  • Verify sprite still runs above/in front of sidebars
  • Verify header still overlaps the sprite

🤖 Generated with Claude Code

backnotprop and others added 2 commits March 15, 2026 21:37
Move TaterSpriteRunning inside the z-0 content wrapper stacking context
so it layers correctly: above sidebars (z-auto) but below the plan
document (z-10). The mobile compat PR (#260) added relative z-0 to the
content wrapper, which trapped the plan at z-0 while the sprite at z-40
floated above everything.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@backnotprop backnotprop merged commit bb4aab2 into main Mar 16, 2026
5 checks passed
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.

1 participant