Skip to content

fix: use checkbox instead of toggle for ToS consent#18014

Merged
tkheyfets merged 9 commits into
mainfrom
do/tos-checkbox
Mar 17, 2026
Merged

fix: use checkbox instead of toggle for ToS consent#18014
tkheyfets merged 9 commits into
mainfrom
do/tos-checkbox

Conversation

@tkheyfets
Copy link
Copy Markdown
Contributor

@tkheyfets tkheyfets commented Mar 16, 2026

Summary

  • Replaced VToggle with native macOS checkbox (Toggle with .toggleStyle(.checkbox)) for the Terms of Service consent in the onboarding flow
  • Moved the checkbox to the leading side of the consent text, which is the standard pattern for consent checkboxes

Original prompt

instead of VToggle for Terms of Service and Privacy Policy let's use our component library checkbox

🤖 Generated with Claude Code


Open with Devin

Replace VToggle with native macOS checkbox (Toggle .checkbox style) for
the Terms of Service consent, placed on the leading side of the text as
is standard for consent checkboxes.
devin-ai-integration[bot]

This comment was marked as resolved.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

@tkheyfets
Copy link
Copy Markdown
Contributor Author

@codex review this PR again — the previous issues have been fixed in commit 0ff8eb8

@tkheyfets
Copy link
Copy Markdown
Contributor Author

@devin review this PR again — the previous issues have been fixed in commit 0ff8eb8

devin-ai-integration[bot]

This comment was marked as resolved.

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@tkheyfets tkheyfets merged commit ca6c3ec into main Mar 17, 2026
2 checks passed
@tkheyfets tkheyfets deleted the do/tos-checkbox branch March 17, 2026 00:12
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