Apply Unstuck design system to live site#65
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR transitions the Unstuck Coach landing page and live demo interface to a simplified content structure and unified design system. The landing page removes feature discovery and complex navigation in favor of a focused hero-to-form-to-benefits flow. A new CSS token system replaces the prior palette and typography, and that system is applied throughout the chat demo components and Hostinger deployment. ChangesLanding page redesign and design system migration
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3b1b5818a
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <form class="stuck-card" action="https://unstuck.kyanitelabs.tech/chat/" method="get" aria-label="Try a stuck sentence"> | ||
| <label for="stuck-sentence">What is the stuck sentence?</label> | ||
| <textarea id="stuck-sentence" name="q" rows="5" placeholder="I need to start this, but my brain will not pick the first move."></textarea> |
There was a problem hiding this comment.
Keep pasted prompts out of query strings
When someone enters or selects a stuck sentence and clicks “Open the coach,” this GET form sends the text as /chat/?q=.... The chat client does not consume q, while the prompt is now exposed in browser history, proxy/access logs, and any page analytics that capture the full URL (the existing tracker records window.location.href as $current_url). This contradicts the new privacy promise that analytics are not used for prompt storage and can leak sensitive pasted messages; pass the draft without putting it in the URL.
Useful? React with 👍 / 👎.
Summary
Verification
Deploy
After merge, deploy /docker/unstuck-coach-live/app on the VPS and verify https://unstuck.kyanitelabs.tech/ plus /chat/.
Need help on this PR? Tag
@codesmithwith what you need. Autofix is disabled.Summary by CodeRabbit
Style
Refactor