Skip to content

Fix IED relay fascia status placement and QA - #128

Merged
masarray merged 8 commits into
mainfrom
agent/ied-relay-icon-industrial-qa
Aug 4, 2026
Merged

masarray merged 8 commits into
mainfrom
agent/ied-relay-icon-industrial-qa

Conversation

@masarray

@masarray masarray commented Aug 4, 2026 •

Copy link
Copy Markdown
Owner

Audit result

The relay PNG was already packaged and reused, but three details were incomplete: the IED Explorer status badge still overlapped the artwork area, a relay-icon shadow remained, and IO List FAT used a different operational-state placement. The QA document also contained machine-local evidence paths.

Changes

  • keep Assets/ied-protection-relay-fascia.png as the shared artwork
  • place LIVE/STOP below the relay in IED Explorer and IO List FAT
  • retain the final PASS badge in the IO List FAT text area
  • remove the relay-icon shadow
  • strengthen the existing 11-test UI contract
  • replace local-only QA paths with repository-auditable documentation
  • update the build invariant from the former 44×44 icon to the new 50×50 layout

Validation

  • XAML parse checks passed
  • Release build passed with zero errors
  • focused UI contracts: 11 passed, 0 failed, 0 skipped
  • Build ARSAS passed
  • Validate IO List Testing passed
  • Validate SV evidence bundles passed
  • Legacy SAS RCB export guard passed
  • regression tests, portable publish, and artifact upload passed

A fresh full-window screenshot at common Windows scaling factors remains a P3 visual evidence item.

@masarray masarray closed this Aug 4, 2026
@masarray masarray reopened this Aug 4, 2026
@masarray
masarray marked this pull request as ready for review August 4, 2026 02:41
@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 186fa26 into main Aug 4, 2026
4 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