Skip to content

feat(inbox): Two-pane layout#1408

Merged
Twixes merged 1 commit intomainfrom
twixes/inbox-two-pane-layout
Apr 2, 2026
Merged

feat(inbox): Two-pane layout#1408
Twixes merged 1 commit intomainfrom
twixes/inbox-two-pane-layout

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented Apr 1, 2026

Problem

Code inbox uses a right sidebar that opens/closes and replaces the whole view for empty states. Cloud has a permanent two-pane split more like an email app, with empty states shown contextually. Let's go for the more robust email-like experience.

Changes

Adding an always-visible resizable left pane (report list) + right pane (detail) when reports exist.

When no reports: full-width view with toolbar + faded skeleton backdrop, gradient overlay, and contextual empty state (welcome with graphs-hog, warming-up with explorer-hog). When reports exist but none selected: mail-hog "Select a report" prompt in right pane.

Signal source config is in a modal instead instead of navigating away to Settings.

Search disabled with tooltip when project has no reports.

@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 7279f12 to ea9068f Compare April 1, 2026 01:51
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch from 36e23e1 to 6d30da1 Compare April 1, 2026 01:51
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch from 6d30da1 to e23878f Compare April 1, 2026 14:26
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from ea9068f to 5f888c8 Compare April 1, 2026 14:26
@Twixes Twixes changed the title feat(inbox): two-pane layout with empty state overlays feat(inbox): Two-pane layout with empty state overlays Apr 1, 2026
@Twixes Twixes changed the title feat(inbox): Two-pane layout with empty state overlays feat(inbox): Two-pane layout Apr 1, 2026
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 5f888c8 to 8fbfe0a Compare April 1, 2026 14:55
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch 2 times, most recently from ac0cf99 to 83433e6 Compare April 1, 2026 15:00
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 8fbfe0a to 78671e0 Compare April 1, 2026 15:00
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch from 83433e6 to b0a601e Compare April 1, 2026 15:12
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 78671e0 to 24ff688 Compare April 1, 2026 15:12
@Twixes Twixes marked this pull request as ready for review April 1, 2026 15:13
@Twixes Twixes requested review from a team April 1, 2026 15:15
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 24ff688 to d0de55c Compare April 1, 2026 16:52
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch 2 times, most recently from 5eca2b3 to 84cdcd4 Compare April 1, 2026 17:08
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from d0de55c to 48cb6bf Compare April 1, 2026 17:08
Copy link
Copy Markdown
Member Author

Twixes commented Apr 2, 2026

Merge activity

  • Apr 2, 1:35 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 2, 1:53 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 2, 1:59 PM UTC: @Twixes merged this pull request with Graphite.

@Twixes Twixes changed the base branch from twixes/inbox-sorting-reviewers to graphite-base/1408 April 2, 2026 13:43
@Twixes Twixes changed the base branch from graphite-base/1408 to main April 2, 2026 13:51
- Resizable left list pane + right detail pane when reports exist
- Full-width view with skeleton backdrop + gradient overlay for empty states
- Welcome state (graphs-hog), warming-up (explorer-hog), select-a-report (mail-hog)
- Inline signal source config modal instead of Settings navigation
- "Back to Inbox" button disabled with tooltip until sources enabled
- Loading skeleton only when sources are configured
- Search disabled with tooltip when no reports
@Twixes Twixes force-pushed the twixes/inbox-two-pane-layout branch from 84cdcd4 to 2140932 Compare April 2, 2026 13:52
@Twixes Twixes merged commit b8d8008 into main Apr 2, 2026
16 checks passed
@Twixes Twixes deleted the twixes/inbox-two-pane-layout branch April 2, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants