Skip to content

P0: show active IED signal tree in correct Browser panel - #126

Merged
masarray merged 18 commits into
mainfrom
fix/p0-browser-fleet-signal-routing
Sep 26, 2026
Merged

masarray merged 18 commits into
mainfrom
fix/p0-browser-fleet-signal-routing

Conversation

@masarray

Copy link
Copy Markdown
Owner

Fixes screenshot-reported multi-IED regression: active AA1E1F06R4 tab shows a different, unselected SCL workspace (Use IED) and blank Data Model.

Root: nested StackLayout with a Repeater; the Repeater itself is a layout child before dynamically generated Browser delegates. Mapping currentIndex directly to fleet.activeIndex can display the wrong slot.

Use a plain Item + persistent Repeater with explicit visible: index === fleet.activeIndex and anchors.fill: parent; preserve each slot's navigation/watchlist. No MMS/SCL protocol changes.

Add --qa-browser-routing rendered Qt QA: slot 0 unselected multi-IED SCL, slot 1 loaded/offline canonical model with a DA, verify tab ↔ view pointer, actual rendered ListView count/geometry, switch both ways, and inactive-slot close/reindex. Run under both Qt and Release Hardening CI.

Do not merge unless exact-head Qt/Release and other PR gates are green. Windows portable ZIP / installer from this fix will be provided for real-IED GUI testing.

@masarray
masarray marked this pull request as ready for review September 26, 2026 13:16
@masarray
masarray merged commit c7238d3 into main Sep 26, 2026
22 checks passed

Copy link
Copy Markdown
Owner Author

Final validation: exact PR head ace34df69a2e76733940230371172ddc2ecf3dbe 11/11 SUCCESS, merged c7238d3a8ff86b30b397164c5fcbed820562b3e0 with zero file changes versus tested head, post-merge 11/11 SUCCESS. Rendered Qt --qa-browser-routing passes active tab/model/list count, offline DA visibility, switching and close/reindex. Windows release artifact ID 10905959885 in merged run 36244651336 (portable ZIP and installer). This addresses screenshot-reported blank signals routing; real AA1E1F06R4 MMS/SCL field verification remains user acceptance.

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