Repository navigation
fix(ui): improve onboarding and unify owner avatars - #627
Conversation
🤖 MERGED
GitHub merged this pull request. No material findings were recorded. Another Agent took this Review. |
|
🤖 Harlan Agent Kit Agent assessment of head Independent review found no credible defect. GitHub Actions passed on this exact head. The service review remains stopped for this head: six Review or Repair tasks were queued, with one free host slot. This is a separate agent assessment, not a Service READY outcome. The shared owner avatar now appears in generic Skill cards, onboarding selections, and watched Repository rows. Actual proxy images loaded in the built app. Phone names wrap cleanly, and Watch occupies y640–684 within the812pxviewport. Saved indexing-off state survives leaving for Email and returning.
The prior production replay proved real Agent OAuth, one-star import, Watch creation, reload persistence, and test-data cleanup. These updated UI screenshots use a controlled local account fixture. No user-facing code changed after capture; the final commit adds accessibility coverage. Confidence95/100 for these fixes. Fresh-account creation, assistive-device playback, full1,000-star import, and actual email delivery remain untested. |






🔗 Linked issue
Follow-up to #626.
❓ Type of change
📚 Description
Repository settings hid the main Watch action on phones and ignored saved indexing choices. Repository rows also lacked the owner avatars shown on Skill cards.
Watch now comes before settings. Saved choices have loading, retry, and save failure messages. Skill cards and Repository rows share one avatar component, with initials if an image fails.
Import updates announce to screen readers, the active step has a semantic label, and Watch removal says “Stop watching”.