Skip to content

Commit ab54834

Browse files
committed
refactor(aria/accordion): underscore pattern props only for internal usage
1 parent 04773d5 commit ab54834

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

src/aria/accordion/accordion.ts

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -79,14 +79,14 @@ export class AccordionPanel {
7979
readonly visible = computed(() => !this._pattern.hidden());
8080

8181
/** The parent accordion trigger pattern that controls this panel. This is set by AccordionGroup. */
82-
readonly accordionTrigger: WritableSignal<AccordionTriggerPattern | undefined> =
82+
readonly _accordionTriggerPattern: WritableSignal<AccordionTriggerPattern | undefined> =
8383
signal(undefined);
8484

8585
/** The UI pattern instance for this panel. */
8686
readonly _pattern: AccordionPanelPattern = new AccordionPanelPattern({
8787
id: this.id,
8888
panelId: this.panelId,
89-
accordionTrigger: () => this.accordionTrigger(),
89+
accordionTrigger: () => this._accordionTriggerPattern(),
9090
});
9191

9292
constructor() {
@@ -98,17 +98,17 @@ export class AccordionPanel {
9898

9999
/** Expands this item. */
100100
expand() {
101-
this.accordionTrigger()?.open();
101+
this._accordionTriggerPattern()?.open();
102102
}
103103

104104
/** Collapses this item. */
105105
collapse() {
106-
this.accordionTrigger()?.close();
106+
this._accordionTriggerPattern()?.close();
107107
}
108108

109109
/** Toggles the expansion state of this item. */
110110
toggle() {
111-
this.accordionTrigger()?.toggle();
111+
this._accordionTriggerPattern()?.toggle();
112112
}
113113
}
114114

@@ -169,13 +169,14 @@ export class AccordionTrigger {
169169
readonly active = computed(() => this._pattern.active());
170170

171171
/** The accordion panel pattern controlled by this trigger. This is set by AccordionGroup. */
172-
readonly _accordionPanel: WritableSignal<AccordionPanelPattern | undefined> = signal(undefined);
172+
readonly _accordionPanelPattern: WritableSignal<AccordionPanelPattern | undefined> =
173+
signal(undefined);
173174

174175
/** The UI pattern instance for this trigger. */
175176
readonly _pattern: AccordionTriggerPattern = new AccordionTriggerPattern({
176177
...this,
177178
accordionGroup: computed(() => this._accordionGroup._pattern),
178-
accordionPanel: this._accordionPanel,
179+
accordionPanel: this._accordionPanelPattern,
179180
element: () => this.element,
180181
});
181182

@@ -292,9 +293,9 @@ export class AccordionGroup {
292293

293294
for (const trigger of triggers) {
294295
const panel = panels.find(p => p.panelId() === trigger.panelId());
295-
trigger._accordionPanel.set(panel?._pattern);
296+
trigger._accordionPanelPattern.set(panel?._pattern);
296297
if (panel) {
297-
panel.accordionTrigger.set(trigger._pattern);
298+
panel._accordionTriggerPattern.set(trigger._pattern);
298299
}
299300
}
300301
});

0 commit comments

Comments
 (0)