Skip to content

Formly elimination - initial version - #70

Open
typhoon41 wants to merge 23 commits into
masterfrom
features/BP-1566-eliminate-formly
Open

Formly elimination - initial version#70
typhoon41 wants to merge 23 commits into
masterfrom
features/BP-1566-eliminate-formly

Conversation

@typhoon41

Copy link
Copy Markdown
Member

typhoon41 added 6 commits June 9, 2026 01:24
Added optional per-component signal switch so we can gracefully move away, form by form, table by table.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

typhoon41 and others added 4 commits July 30, 2026 17:33
The generated markup emits a plain aria-labelledby attribute, so a custom
component's @input() ariaLabelledby never receives it. Document the input
alias (and the host-read alternative) before re-binding it on the inner
control.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Ports WSR-5653 option grouping to the signals select path. Master added
grouping against the inline select block in the signals template, which
this branch replaced with AngularSignalsSelectHtmlHelperExtensions, so the
conflict resolution keeps the branch structure and re-implements grouping
there:

- fixed options carry a localized, escaped group;
- groupProperty is emitted in both OptionsConfiguration branches and passed
  to sortOptions, only when a group key is configured;
- selects, multi-selects and autocompletes render mat-optgroup wrappers
  (autocomplete groups its filtered options), with ungrouped options
  rendered in place;
- radio groups and multi-checkboxes keep the group value on each option but
  generate no grouping members, since they have no optgroup equivalent.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

3 participants