Skip to content

fix(mobile): match linked-account rows to other profile cards - #5361

Merged
iscekic merged 1 commit into
mainfrom
linked-accounts-card-c596
Aug 19, 2026
Merged

fix(mobile): match linked-account rows to other profile cards#5361
iscekic merged 1 commit into
mainfrom
linked-accounts-card-c596

Conversation

@iscekic

@iscekic iscekic commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

What

Linked-account rows on the mobile profile screen now use the same ConfigureRow card as Agents, Reviews, Organization, and App.

Why

The WorkOS (and every other provider) row used a custom layout: a bare key icon and the raw provider id (workos). That broke the visual rhythm of the rest of the screen.

How

  • Render each linked provider with ConfigureRow (tinted icon tile, title, email subtitle).
  • Map known provider ids to the same display names as web (Enterprise SSO, GitHub, Google, …).
  • Leave the row inert (no chevron): there is no linked-account detail screen on mobile.

Visual

Before: bare icon + capitalized slug, no tile.
After: boxed tinted icon + proper name + email, same card chrome as the other rows.

The profile linked-accounts list used a custom row with a bare icon
and the raw provider id. Render each provider with ConfigureRow and
the same display names as web.
@iscekic iscekic self-assigned this Aug 19, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the mobile profile linked-account row restyle onto ConfigureRow; labels match web provider names and inert-row behavior is correct.

Files Reviewed (1 files)
  • apps/mobile/src/components/profile-screen.tsx

Reviewed by grok-4.6 · Input: 84.2K · Output: 5.2K · Cached: 195.6K

Review guidance: REVIEW.md from base branch main

@iscekic iscekic added the human-ready The PR is ready for human review. label Aug 19, 2026

@eshurakov eshurakov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linked-account rows now use ConfigureRow and the same provider labels as web. Inert (no chevron) is the right call.

@iscekic
iscekic merged commit 1db4d5c into main Aug 19, 2026
18 checks passed
@iscekic
iscekic deleted the linked-accounts-card-c596 branch August 19, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

human-ready The PR is ready for human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants