docs(main-detail-container): move configuration controls into card#2182
docs(main-detail-container): move configuration controls into card#2182spike-rabbit wants to merge 4 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors the settings forms in the main-detail container examples to use si-form-item components and ngModel bindings, importing FormsModule and SiFormItemComponent in the process. Feedback highlights an invalid [ngModel] property binding in si-main-detail-container-block.html that will cause a compilation error, as well as violations of the Siemens UX writing guidelines regarding capitalization for the 'Resizable Parts' labels, which should be in sentence case ('Resizable parts').
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
42c006a to
c655c9b
Compare
Aligns with many other examples we have.
21dd0ba to
d87adbe
Compare
Aligns with many other examples we have.
Documentation.
Examples.
Dashboards Demo.
Playwright report.
Coverage Reports: