Skip to content

Conversation

@andrewseguin
Copy link
Contributor

// Before
<div ngAccordionGroup [(values)]="openedPanels">
  <h2>
    <button ngAccordionTrigger value="a11y">
      Title
    </button>
  </h2>
  <div ngAccordionPanel value="a11y">
    <ng-template ngAccordionContent>
      Content
    </ng-template>
  </div>
// After
<div ngAccordionGroup [(expandedPanels)]="openedPanels">
  <h2>
    <button ngAccordionTrigger panelId="a11y">
      Title
    </button>
  </h2>
  <div ngAccordionPanel panelId="a11y">
    <ng-template ngAccordionContent>
      Content
    </ng-template>
  </div>

@andrewseguin andrewseguin requested a review from a team as a code owner November 10, 2025 22:26
@andrewseguin andrewseguin requested review from tjshiu and wagnermaciel and removed request for a team November 10, 2025 22:26
@andrewseguin andrewseguin added the target: rc This PR is targeted for the next release-candidate label Nov 10, 2025
@angular-robot angular-robot bot requested a review from tjshiu November 11, 2025 12:30
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Nov 12, 2025
@andrewseguin andrewseguin merged commit 221402f into angular:main Nov 12, 2025
22 of 25 checks passed
@andrewseguin
Copy link
Contributor Author

This PR was merged into the repository. The changes were merged into the following branches:

andrewseguin added a commit that referenced this pull request Nov 12, 2025
…32295)

* fix(aria/accordion): rename value to panelId for trigger and panel

* refactor: fix tests

* refactor: fix tests

(cherry picked from commit 221402f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants