Skip to content

refactor(ui): the scanline reveal is for chart loading only — take it off the Workspace stage, thread and briefing; pages and lists load with skeleton placeholders #2540

Description

@vybe

Ruling

Operator, 2026-09-06 (Workspace review of the v0.9.5 design): "make sure this animation of horizontal loading that we are now using across the workspace is not used for loading entire pages — that's not the right animation. This animation should only be used for chart loading, not for anything else."

This amends design-system principle 12 (docs/memory/design-system.md: "One loading motion app-wide: scanline beam + wipe-in reveal; no bespoke spinners or skeletons"). The scanline beam is the chart-loading motion. Pages, panels, lists and message threads load with skeleton placeholders keyed on "no data yet" (the pattern PortalAgentPage.vue and the #2159 sidebar already use). Principles 13 and 14 are unchanged: first load shows a placeholder, background refresh is invisible, loading means no data yet.

Where it is wrong today (origin/dev)

  • src/frontend/src/views/Portal.vueScanlineReveal wraps the Workspace stage
  • src/frontend/src/components/portal/PortalConversation.vue — wraps the thread history
  • src/frontend/src/components/portal/PortalBriefing.vue — wraps the briefing hints

All three came in with #2163's first-load criterion of 2026-09-02, which this ruling reverses.

Acceptance Criteria

Related: #1921, #2163, ent#253, ent#475 (its Files-panel criterion), ent#449, ent#245 (the primitive), #1430 (epic).

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions