🏗️ Vanguard: [Component/System] Parity Integration#291
Conversation
Co-authored-by: tedd <493224+tedd@users.noreply.github.com>
|
đź‘‹ Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a đź‘€ emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
There was a problem hiding this comment.
Pull request overview
This PR aligns the Blazor declarative demo’s left navigation layout with the programmatic demo by explicitly setting the navigation expander width, supporting the project’s cross-paradigm parity goal.
Changes:
- Set
Width="20"on theTuiExpandernavigation pane inHome.razorto match the programmatic implementation. - Documented the parity adjustment in
.jules/vanguard.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/Tedd.TUI.Demo.Blazor/Pages/Home.razor |
Adds explicit Width="20" to the left navigation TuiExpander to match programmatic layout. |
.jules/vanguard.md |
Adds a changelog entry describing the expander width parity update. |
đź’ˇ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## 2026-06-11 - TabControl Selection Synchronization Parity | ||
| **Observation:** The bifurcated architecture correctly synced navigation from the TreeView to the TabControl across all paradigms. However, bidirectional synchronization—where navigating via the TabControl updates the TreeView state—was absent in the Blazor deployment targets (Home.razor and Programmatic.razor). | ||
| **Strategic Action:** Implemented identical `SelectionChanged` logic within the TabControl across the Blazor deployments to map active tabs back to the TreeView context. Introduced derived `@bind-IsSelected` boolean bindings on Razor TreeViewItems, driven from `_selectedTabIndex`, to keep navigation state synchronized between the TreeView and TabControl. | ||
| ## 2025-02-24 - Structural Sync for TuiExpander |
đź’ˇ Objective: Address structural layout anomalies in declarative component setups missing exact programmatic equivalents.
🎯 Execution: Modified
TuiExpanderinHome.razorto applyWidth="20", structurally matching its programmatic counterpart inProgrammatic.razor.đź“Š Functional Impact: Enhances rendering layout equivalence directly establishing an exact cross-platform synergy per the 1:1 architectural rendering mandate.
🔬 Verification Protocol: Compile applications via
dotnet buildand visually confirm Left Dock navigation menu dimensions uniformly map to20units in HTML (Blazor) contexts against the Terminal context equivalents.PR created automatically by Jules for task 13579875826879865866 started by @tedd