UI refinements: nav legibility, hover states, card polish - #7
Merged
Conversation
- Nav links: replace invisible outlined-white text with solid ink, add wash/border hover and full-ink active state - Nav hamburger + drawer: same fix — solid ink fill, add hover states - Secondary btn: invert to ink fill on hover for clear affordance - Input fields: upgrade to 1.5px solid ink border, offset box-shadow on focus - Splash feature cards + steps: use solid ink on h3/step-label (outlined graffiti style is illegible at 1.25–1.3rem) - Cards and explore list: upgrade border to 1.5px ink, add 3px offset box-shadow on hover for consistent depth language
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe global stylesheet updates form controls, buttons, navigation controls, and homepage cards. The changes add stronger borders, focus and hover feedback, active states, transitions, and solid ink-colored text. ChangesVisual styling updates
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
zowskyy
marked this pull request as ready for review
August 21, 2026 17:34
zowskyy
pushed a commit
that referenced
this pull request
Aug 22, 2026
…s timestamps MINOR Bug #7: Ambient Status ISO String Timestamp Comparison - Changed expiresAt from ISO string to Unix milliseconds - Numeric comparison is more explicit and robust than lexicographic string comparison - Eliminates millisecond-level edge cases from string ordering - Updated AmbientStatus interface to reflect numeric timestamp - Updated setAmbientStatus, getAmbientStatus, getAmbientStatuses functions - Updated test to use Unix milliseconds instead of ISO strings MINOR Bug #8: Silent JSON.parse Failures - Added console.warn logging when JSON.parse fails in: * app/src/app/[handle]/page.tsx (resolveTopEight function) * app/src/app/(platform)/vibe/page.tsx (2 locations) * app/src/lib/collections.ts (getCollectionMembers function) - Logs now surface corrupted document_json to monitoring/debugging - Prevents silent data loss and makes issues discoverable Test Results: - All 9 ambient status tests passing - Changes enable better visibility into data corruption Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018V1rEEt5QTC2ww5ZikioWZ
zowskyy
pushed a commit
that referenced
this pull request
Aug 22, 2026
Bug #7 fix follow-up: Updated the race condition test in bugs.test.ts to use Unix milliseconds instead of ISO string for the expired timestamp, matching the updated getAmbientStatus function behavior. All 251 tests passing. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018V1rEEt5QTC2ww5ZikioWZ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
color: var(--paper)(white fill) with a black stroke — effectively invisible on the white background. Fixed to solidvar(--ink)with a wash/border hover and full-ink active state..btn: inverts to ink background on hover so the affordance is clear.var(--line)border to a crisp1.5px solid var(--ink)border; focus now shows a3px 3px 0offset box-shadow instead of the default browser outline.h3and.splash-step-labelat 1.25–1.3rem were using the outlined graffiti style (white fill, black stroke) — illegible at that size. Changed to solidvar(--ink).1.5px solid var(--ink), hover adds a3px 3px 0box-shadow for consistent depth language across the page.Generated by Claude Code
Summary by CodeRabbit