tweak signals

Author: adolgachev

goldens/aria/accordion/index.api.md

@@ -61,6 +61,7 @@ export class AccordionTrigger implements OnInit {
     // (undocumented)
     ngOnInit(): void;
     readonly panel: _angular_core.InputSignal<AccordionPanel>;
+    readonly panelId: _angular_core.Signal<string>;
     _pattern: AccordionTriggerPattern;
     toggle(): void;
     // (undocumented)

src/aria/private/accordion/accordion.ts

@@ -158,7 +158,7 @@ export interface AccordionTriggerInputs
 /** A pattern controls the expansion state of an accordion. */
 export class AccordionTriggerPattern implements ListNavigationItem, ListFocusItem, ExpansionItem {
   /** A unique identifier for this trigger. */
-  readonly id: SignalLike<string> = () => this.inputs.id();
+  readonly id: SignalLike<string>; // set from inputs
 
   /** A reference to the trigger element. */
   readonly element: SignalLike<HTMLElement> = () => this.inputs.element()!;
@@ -167,13 +167,13 @@ export class AccordionTriggerPattern implements ListNavigationItem, ListFocusIte
   readonly expandable: SignalLike<boolean> = () => true;
 
   /** Whether the corresponding panel is expanded. */
-  readonly expanded: WritableSignalLike<boolean>;
+  readonly expanded: WritableSignalLike<boolean>; // set from inputs
 
   /** Whether the trigger is active. */
   readonly active = computed(() => this.inputs.accordionGroup().inputs.activeItem() === this);
 
   /** Id of the accordion panel controlled by the trigger. */
-  readonly controls = computed(() => this.inputs.accordionPanelId());
+  readonly controls: SignalLike<string>; // set from inputs
 
   /** The tabindex of the trigger. */
   readonly tabIndex = computed(() =>
@@ -191,7 +191,9 @@ export class AccordionTriggerPattern implements ListNavigationItem, ListFocusIte
   );
 
   constructor(readonly inputs: AccordionTriggerInputs) {
+    this.id = inputs.id;
     this.expanded = inputs.expanded;
+    this.controls = inputs.accordionPanelId;
   }
 
   /** Opens the accordion panel. */