Skip to content

Conversation

@marcodejongh
Copy link
Owner

Add a segmented control at the top of the log ascent form that allows
users to choose between logging an "Ascent" (flash/send) or an "Attempt".
When "Attempt" is selected, the status is always set to "attempt"
regardless of the attempt count.

Add a segmented control at the top of the log ascent form that allows
users to choose between logging an "Ascent" (flash/send) or an "Attempt".
When "Attempt" is selected, the status is always set to "attempt"
regardless of the attempt count.
@vercel
Copy link

vercel bot commented Dec 31, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
boardsesh Ready Ready Preview, Comment Dec 31, 2025 7:59am

@claude
Copy link

claude bot commented Dec 31, 2025

Claude Review

No significant issues found.

Minor notes:

  • logascent-form.tsx:146: Uses inline style={{ marginBottom: '16px' }} - per CLAUDE.md, prefer avoiding the style property and using design tokens.
  • logascent-form.tsx:24-26: getAscentStatus is now only called from within getTickStatus - could be inlined, but this is purely stylistic.

Change the drawer height from 90% to 100% to fill all available space.
@marcodejongh marcodejongh merged commit 3ba7af6 into main Dec 31, 2025
5 of 6 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.

3 participants