Repository navigation
Redesign every screen, the palette, and running subagents - #106
Merged
Merged
Conversation
…f, and pairing copy
…rast; tidy execution cards
…hen switching servers
…eject note, delete context, one chevron direction, tighter code headers, shared review button
…, model, variant without chip borders
…ed permission details actions with focus return, settings dot on the gear, subagent and diff action chevrons, transcript image grid
…casing, tighter copy icon, session dot by the chevron, platform-aligned header subtitle
…d as one phrase; capsule buttons with filled secondary and danger variants
…h the native root background
… and needs-you states, elapsed time, and no result before it exists
…et lists Needs you, Working, and Finished with open and confirmed stop
…fix, and say back returns to the parent
…ments never become results, Stop reconciles and stays Stopping, stale monitors keep last-known rows, pressed buttons keep label contrast
…8 for the native background change
…RAP below 30; test a failed stop
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
A design pass over every screen, sheet, and menu. Three reviewers from different model providers audited the app from source, then from phone-sized screenshots of a web preview. The owner supplied screenshots of the official OpenCode iOS app as the quality bar. Its visual patterns were adopted where they fit existing features; its other features were left out.
palette.selectedfill marks every chosen tab, option, and row. The old selected fill matched the card and was invisible.theme.tsalso gains ripple, floating-shadow, and icon-tile tokens.#11131abackground,#1a1d27card). Working is one blue everywhere, as the spec says. Every text color clears 4.5:1 on each surface it is used on.The native root background in
app.config.tsnow matches the palette. That changes the native fingerprint, so this needs new native builds and can't ship as an OTA update to 0.1.5 builds. The preview runtime moves to 0.1.6, with iOS build 9 and Android version code 8.Deliberately not changed:
Verification
pnpm check: lint, typecheck, 1527 tests in 114 suites, and iOS and Android Hermes exports.pnpm native:doctor: 18 of 18 checks pass.pnpm crapscores every changed function below 30. The highest score is 30.0, ininstallNotificationPairing, which this branch does not touch.d71a46aon both platforms and are still running. Later commits change JavaScript only, so they keep the same native fingerprint.docs/UX-REVIEW.mdlists what to check on devices: large text, VoiceOver and TalkBack, keyboard, Android header alignment, the iOS 26 header, hit-slop targets, and Stop against a real server. No probe is recorded indocs/COMPATIBILITY.mdyet.TODO.mdreflect the resulting behavior.docs/UX-REVIEW.mdrecords the review and its device follow-up.TODO.mdis unchanged, because its token and theming items stay open until device verification.mobile-updatesenvironment:MOBILE_RUNTIME_VERSIONand both native hashes frompnpm mobile:runtime. Until then, Publish mobile update will refuse to publish.Council review
Rounds: 7
Rounds 5 to 7 widened the review to the composer, buttons, color, and running subagents. All three reviewers were satisfied after round 7. GPT's round 7 score is from its confirmation after the announcement fix.