Skip to content

feat(sig): Report list toolbar as core control#1532

Open
Twixes wants to merge 1 commit intotwix/inbox-row-visualsfrom
twix/inbox-selection-ux
Open

feat(sig): Report list toolbar as core control#1532
Twixes wants to merge 1 commit intotwix/inbox-row-visualsfrom
twix/inbox-selection-ux

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented Apr 7, 2026

Problem

The inbox list toolbar introduced in #1513 added controls only for bulk operations. But the same controls have been missing for individual reports too. In the Cloud version, we had them under a dropdown in the report pane itself. I considered doing the same here, but there's no point when we have a great toolbar - the toolbar should be the individual report controls as well.

At the same time, checkboxes take away from the content when shown always - and we don't need them by default for most user interactions here. (Ofc will validate this, but at least I think that's a safe assumption for now that the individual report level is the primary one.)

Changes

Making the report list toolbar always-on, with intuitive logic for selection state synced with the currently-viewed-report state. Unifies the selection state, so that "currently viewed" and "selected" are the same thing. Checkboxes only show up when multiple reports are selected. For multi-select, you can use the "Select all" checkbox or standard desktop Shift/Cmd+click patterns. See in action:

Copy link
Copy Markdown
Member Author

Twixes commented Apr 7, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from c53762d to f96bd7a Compare April 7, 2026 19:57
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 67f9a59 to af669e3 Compare April 7, 2026 19:57
@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from f96bd7a to 3971708 Compare April 7, 2026 20:00
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from af669e3 to 0e246ba Compare April 7, 2026 20:00
@Twixes Twixes changed the title Rework inbox bulk selection UX s Apr 7, 2026
@Twixes Twixes changed the title s feat(sig): Always show report controls Apr 7, 2026
@Twixes Twixes changed the title feat(sig): Always show report controls feat(sig): Make report list toolbar always-on Apr 7, 2026
@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from 3971708 to ae4d475 Compare April 7, 2026 20:41
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 0e246ba to 8f8f5fb Compare April 7, 2026 20:41
@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from ae4d475 to 57e72a2 Compare April 7, 2026 21:18
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 8f8f5fb to 200faf9 Compare April 7, 2026 21:18
@oliverb123 oliverb123 force-pushed the twix/inbox-row-visuals branch from 57e72a2 to 8e02e34 Compare April 7, 2026 21:51
@oliverb123 oliverb123 force-pushed the twix/inbox-selection-ux branch from 200faf9 to 7f9bc81 Compare April 7, 2026 21:52
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 7f9bc81 to ea177bc Compare April 8, 2026 06:16
@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from 8e02e34 to da7746d Compare April 8, 2026 06:16
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from ea177bc to e0e00eb Compare April 8, 2026 06:55
@Twixes Twixes changed the title feat(sig): Make report list toolbar always-on feat(sig): Report list toolbar as core control Apr 8, 2026
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from e0e00eb to 5a7e2ab Compare April 8, 2026 08:38
@Twixes Twixes force-pushed the twix/inbox-row-visuals branch from da7746d to 41031df Compare April 8, 2026 08:38
@Twixes Twixes marked this pull request as ready for review April 8, 2026 08:40
@Twixes Twixes requested a review from a team April 8, 2026 08:40
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 5a7e2ab to 5fb9697 Compare April 8, 2026 09:36
@Twixes Twixes force-pushed the twix/inbox-selection-ux branch from 5fb9697 to 7b212dd Compare April 8, 2026 09: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.

3 participants