Skip to content

TlonBotSetupPaneView: use arvos.mp4#5822

Open
jamesacklin wants to merge 1 commit into
stagingfrom
ja/arvos-waiting
Open

TlonBotSetupPaneView: use arvos.mp4#5822
jamesacklin wants to merge 1 commit into
stagingfrom
ja/arvos-waiting

Conversation

@jamesacklin
Copy link
Copy Markdown
Member

Traps arvos.mp4 in a circle; works with light mode and dark mode.

Simulator.Screen.Recording.-.iPhone.17.-.2026-05-05.at.13.50.20.mov

@jamesacklin jamesacklin requested a review from latter-bolden May 5, 2026 17:52
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@dnbrwstr
Copy link
Copy Markdown
Contributor

dnbrwstr commented May 6, 2026

I like the animation, layout feels a little odd:

  • Log out button jumps around in video
  • Content isn't centered vertically (I think it's being pushed down by the logout button, but feels off visually)
  • Circle bg feels off brand, not visual language we use elsewhere. Maybe make both the video and the bg bigger and trap in a rounded rect?
  • Maybe consider $secondaryText for the text color? Or maybe a different text weight? There's something about the text that feels inconsistent to me

Base automatically changed from lb/tlonbot-revival to staging May 8, 2026 15:49
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.

2 participants