Skip to content

fix(session-ui): align provider error icon with first line - #52859

Merged
Hona merged 2 commits into
v2from
error-icon-alignment
Oct 6, 2026
Merged

Hona merged 2 commits into
v2from
error-icon-alignment

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Aligns the provider error icon with the first line of the message instead of the middle of the whole block.

- <div class="flex w-full min-w-0 items-center gap-2">
+ <div class="flex w-full min-w-0 items-start gap-2">
    <Icon name="outline-hexagonal-warning" class="shrink-0 text-v2-state-fg-danger" />
  • The icon stays 16px, the same as the 16px line height and the retry card icon. When a retry turns into an error, the icon stays the same size and position.
  • error-card.stories.tsx shows one short and one wrapped provider error. The component test checks that the icon center equals the first line's center in both errors, and that the long error wraps.

Real app timeline (setupTimeline e2e harness):

Before After
Before: the icon sits between the two lines of the wrapped error After: the icon is centered on the first line of the wrapped error

Requested by: @iamdavidhill

@Hona
Hona force-pushed the error-icon-alignment branch from a09a601 to cb62309 Compare October 6, 2026 02:44
@Hona
Hona merged commit 9daf46b into v2 Oct 6, 2026
11 checks passed
@Hona
Hona deleted the error-icon-alignment branch October 6, 2026 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants