fix(aria/ui-patterns): internal conformance fixes

Author: wagnermaciel

src/aria/ui-patterns/accordion/accordion.ts

@@ -116,8 +116,6 @@ export class AccordionTriggerPattern {
     this.id = inputs.id;
     this.element = inputs.element;
     this.value = inputs.value;
-    this.accordionGroup = inputs.accordionGroup;
-    this.accordionPanel = inputs.accordionPanel;
     this.expansionControl = new ExpansionControl({
       ...inputs,
       expansionId: inputs.value,
@@ -148,10 +146,10 @@ export class AccordionTriggerPattern {
   /** The keydown event manager for the accordion trigger. */
   keydown = computed(() => {
     return new KeyboardEventManager()
-      .on(this.prevKey, () => this.accordionGroup().navigation.prev())
-      .on(this.nextKey, () => this.accordionGroup().navigation.next())
-      .on('Home', () => this.accordionGroup().navigation.first())
-      .on('End', () => this.accordionGroup().navigation.last())
+      .on(this.prevKey, () => this.inputs.accordionGroup().navigation.prev())
+      .on(this.nextKey, () => this.inputs.accordionGroup().navigation.next())
+      .on('Home', () => this.inputs.accordionGroup().navigation.first())
+      .on('End', () => this.inputs.accordionGroup().navigation.last())
       .on(' ', () => this.expansionControl.toggle())
       .on('Enter', () => this.expansionControl.toggle());
   });
@@ -162,7 +160,7 @@ export class AccordionTriggerPattern {
       const item = this._getItem(e);
 
       if (item) {
-        this.accordionGroup().navigation.goto(item);
+        this.inputs.accordionGroup().navigation.goto(item);
         this.expansionControl.toggle();
       }
     });
@@ -183,7 +181,7 @@ export class AccordionTriggerPattern {
     const item = this._getItem(event);
 
     if (item && this.inputs.accordionGroup().focusManager.isFocusable(item)) {
-      this.accordionGroup().focusManager.focus(item);
+      this.inputs.accordionGroup().focusManager.focus(item);
     }
   }
 
@@ -193,7 +191,8 @@ export class AccordionTriggerPattern {
     }
 
     const element = e.target.closest('[role="button"]');
-    return this.accordionGroup()
+    return this.inputs
+      .accordionGroup()
       .items()
       .find(i => i.element() === element);
   }

src/aria/ui-patterns/tree/combobox-tree.ts

@@ -21,7 +21,7 @@ export class ComboboxTreePattern<V>
   implements ComboboxTreeControls<TreeItemPattern<V>, V>
 {
   /** Whether the currently focused item is collapsible. */
-  isItemCollapsible = () => this.activeItem()?.parent() instanceof TreeItemPattern;
+  isItemCollapsible = () => this.inputs.activeItem()?.parent() instanceof TreeItemPattern;
 
   /** The ARIA role for the tree. */
   role = () => 'tree' as const;
@@ -94,7 +94,7 @@ export class ComboboxTreePattern<V>
   collapseItem = () => this.collapse();
 
   /** Whether the specified item or the currently active item is expandable. */
-  isItemExpandable(item: TreeItemPattern<V> | undefined = this.activeItem()) {
+  isItemExpandable(item: TreeItemPattern<V> | undefined = this.inputs.activeItem()) {
     return item ? item.expandable() : false;
   }
 

src/aria/ui-patterns/tree/tree.ts

@@ -84,6 +84,30 @@ export class TreeItemPattern<V> implements ExpansionItem {
     return this.tree().value().includes(this.value()) ? this.tree().currentType() : undefined;
   });
 
+  /** A unique identifier for this item. */
+  id: SignalLike<string>;
+
+  /** The value of this item. */
+  value: SignalLike<V>;
+
+  /** A reference to the item element. */
+  element: SignalLike<HTMLElement>;
+
+  /** Whether the item is disabled. */
+  disabled: SignalLike<boolean>;
+
+  /** The text used by the typeahead search. */
+  searchTerm: SignalLike<string>;
+
+  /** The tree pattern this item belongs to. */
+  tree: SignalLike<TreePattern<V>>;
+
+  /** The parent item. */
+  parent: SignalLike<TreeItemPattern<V> | TreePattern<V>>;
+
+  /** The children items. */
+  children: SignalLike<TreeItemPattern<V>[]>;
+
   constructor(readonly inputs: TreeItemInputs<V>) {
     this.id = inputs.id;
     this.value = inputs.value;