Repository navigation
fix(studio): native dark Advanced controls and consistent dedicated Studio/Inno icon - #194
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Final merged handoff — 2026-10-16 native lab UX / iconMerged PR #194 into What users now get (merged #192/#193/#194):
Verified installer handoff: Windows Native Preview Actions run Physical acceptance still required: Operator may install preview without reflashing already compatible verified ESP32-P4. Validate native Windows window/taskbar/installer icon; right-click active current/voltage ABC fields (Equal, Balance) while SMV running and verify Pcap waveform/phasor and atomic SET batch outcome; PTP switch Source ON/OFF with device TX counters and independent Analyzer PTP capture, Follower searching/receive lock only with an actual external GM, AUTO/forced smpSynch simulated on-wire flagged correctly; use Copy diagnostics on failure. Do not claim hardware PTP-to-SV epoch lock or certified 61850-9-3 conformance (M2/M3 open). |
Operator's Windows UI issue
The physical operator reports black/unreadable text in Advanced and generic icon branding.
mainalready gained real PTP Source/Follower/Monitor + smpSynch lab simulation UI and ABC right-click via merged PR #192/#193; do NOT duplicate or revert those changes. Older PR #190/#191 remain overlapping and are not a safe integration target.Changes
assets/studio_iconartwork already reviewed in PR feat(studio): dedicated phasor icon for native GUI and Inno Setup #190; add its validated, multi-resolutionARStackStudio.icoblob tomainsource. CMakeconfigure_file(COPYONLY)puts that exact same icon in the canonical Windows build output, existing Windows PE.rcembeds it inARStackStudio.exe, and Inno's existingSetupIconFile={#STAGE_DIR}\\ARStackStudio.icouses that same staged icon. Qt embeds the matching 192/512 px PNGs as resources and setsQGuiApplication::setWindowIcon. The public Workbench icon and stable v0.1.1 release stay unchanged.StudioDarkCheckBoxused by Timing advanced/VLAN/Pdelay and Waveform saturation controls, removing black-on-dark native checkbox text.StudioDarkComboBoxfor the Signals SCL selector and profile family, plus Device manual recovery port; adjust model delegate tocombo.textAt(index)so list and QAbstractItemModel textRole are supported in one native dark dropdown.Acceptance
Require exact PR-head Qt/QML compilation, harness/lifecycle, native Windows PE/Inno installer clean installation/upgrade/uninstall, and final artifact icon provenance check before merge. Physical screenshot readability / right-click PTP and 4I4V acceptance still requires operator. No changes to experimental M3 PR #166 or public stable release.