Skip to content

feat(form): support Angular's signal forms with new si-form-field#2272

Merged
spliffone merged 1 commit into
mainfrom
feat/forms/form-field-component
Jul 7, 2026
Merged

feat(form): support Angular's signal forms with new si-form-field#2272
spliffone merged 1 commit into
mainfrom
feat/forms/form-field-component

Conversation

@spike-rabbit

@spike-rabbit spike-rabbit commented Jul 3, 2026

Copy link
Copy Markdown
Member

Introduces a new component called si-form-field which is built around Angular's signal forms.
This initial version only supports native browser inputs. Currently missing:

  • integration of the error resolution mechanism
  • label in a separate column
  • fieldset combination with fieldset
  • documentation

Documentation.
Examples.
Dashboards Demo.
Playwright report.

Coverage Reports:

Code Coverage

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new SiFormFieldComponent and its configuration provider to support Angular signal forms, along with an example form and corresponding E2E tests. The feedback focuses on adhering to the repository style guide by replacing state-propagating effect() calls with computed() signals in the form field component, and aligning user-facing error messages and checkbox labels with the Siemens UX writing guidelines (e.g., using telegram style, avoiding abbreviations, and removing trailing punctuation). E2E test assertions should also be updated to match these revised text labels.

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.

Comment thread projects/element-ng/form/si-form-field/si-form-field.component.ts Outdated
Comment thread projects/element-ng/form/si-form-field/si-form-field.component.ts
Comment thread src/app/examples/si-signal-form/si-signal-form.ts Outdated
Comment thread src/app/examples/si-signal-form/si-signal-form.html Outdated
Comment thread playwright/e2e/element-examples/si-signal-form.spec.ts Outdated
@spike-rabbit spike-rabbit force-pushed the feat/forms/form-field-component branch from fcacf86 to 860c76e Compare July 3, 2026 09:19
@spike-rabbit spike-rabbit marked this pull request as ready for review July 3, 2026 09:28
@spike-rabbit spike-rabbit requested review from a team as code owners July 3, 2026 09:28
@spike-rabbit spike-rabbit force-pushed the feat/forms/form-field-component branch from 860c76e to 2c2dff2 Compare July 3, 2026 09:40
Comment thread projects/element-ng/form/si-form-field/si-form-field.component.spec.ts Outdated
Comment thread projects/element-ng/form/si-form-field/si-form-field.component.ts Outdated
Comment thread projects/element-ng/form/si-form-field/si-form-field.component.ts
Comment thread projects/element-ng/form/si-form-field/si-form-field.component.ts
Comment thread projects/element-ng/form/si-form-field/si-form-field.provider.ts Outdated
@spike-rabbit spike-rabbit force-pushed the feat/forms/form-field-component branch from 2c2dff2 to 1b8950d Compare July 3, 2026 10:37
@spike-rabbit spike-rabbit requested a review from spliffone as a code owner July 3, 2026 10:37
Comment thread src/app/examples/si-signal-form/si-signal-form.ts Outdated
Comment thread src/app/examples/si-signal-form/si-signal-form.html
@spike-rabbit spike-rabbit mentioned this pull request Jul 7, 2026
5 tasks
@spike-rabbit spike-rabbit force-pushed the feat/forms/form-field-component branch 2 times, most recently from f1312ba to 124e630 Compare July 7, 2026 08:40
Introduces a new component called `si-form-field` which is built around
Angular's signal forms.
This initial version only supports native browser inputs.
Currently missing:
- integration of the error resolution mechanism
- label in a separate column
- fieldset combination with fieldset
- documentation
@spike-rabbit spike-rabbit force-pushed the feat/forms/form-field-component branch from 124e630 to 5f0f052 Compare July 7, 2026 09:37
@spliffone spliffone enabled auto-merge July 7, 2026 10:04
@fh1ch fh1ch dismissed chintankavathia’s stale review July 7, 2026 10:16

Adjusted as requested.

@spliffone spliffone added this pull request to the merge queue Jul 7, 2026
Merged via the queue into main with commit 98fa995 Jul 7, 2026
15 checks passed
@spliffone spliffone deleted the feat/forms/form-field-component branch July 7, 2026 10:29
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.

4 participants