Skip to content

Unify P0 visual system and Live Monitor filter hierarchy - #204

Merged
masarray merged 22 commits into
mainfrom
polish/p0-visual-system-unification
Aug 17, 2026
Merged

Unify P0 visual system and Live Monitor filter hierarchy#204
masarray merged 22 commits into
mainfrom
polish/p0-visual-system-unification

Conversation

@masarray

@masarray masarray commented Aug 17, 2026

Copy link
Copy Markdown
Owner

P0 — Visual System Unification

Presentation-only cleanup to make ARSAS read as one engineering workstation across Explorer, Live Monitor, Event Log, Alarm, GOOSE, and Diagnostics.

Scope

  • Introduce one compact five-level typography contract (WorkspaceTitle, WorkspaceSubtitle, SectionTitle, BodyText, Caption, MicroLabel).
  • Add shadow-free WorkspaceCard and reusable SearchSurface for main-window workspaces while leaving dialog/card styles untouched.
  • Normalize dense grid typography/density for engineering scan paths.
  • Unify XAML navigation and runtime SasOperationalUiPolicy styling so the visual contract does not change after Loaded; responsive MainWindowNavigationLayoutFix remains final geometry authority at 990/900/720 px.
  • Remove nav font-size jitter on hover/press; use a clean internal keyboard focus ring instead of the default dotted focus visual.
  • Make Global Live search the primary tool. Per-column filters now start collapsed and expand/collapse with a Filters button, while continuing to AND with global search.
  • Shorten verbose workspace subtitles so the data remains the hierarchy.

Safety boundary

This PR does not modify IEC 61850 acquisition, RCB planning, SOE/event semantics, control operations, Alarm state machine, Services/*, or the ARIEC61850 engine lock. Engine remains pinned to becda399b4a3ae34831215fc915798b4f846c1be (PR #81).

Regression coverage

Adds VisualSystemP0RegressionTests and updates the existing process-value presentation wording guard to lock typography/surface tokens, stable navbar behavior, progressive Live Monitor filters, blue/slate state semantics, and the reviewed immutable engine pin.

Temporary patch tooling was removed before merge; final diff contains five presentation files plus two regression-test files.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@masarray
masarray merged commit 5ea1c69 into main Aug 17, 2026
3 checks passed
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.

1 participant