[EuiFilterButton] Don't infer aria-expanded from hasActiveFilters

[DevLikhith5]

Summary

Closes #9692.

EuiFilterButton was using hasActiveFilters as a fallback for aria-expanded when rendered as a collapsible chevron button:

const isExpanded = isCollapsible && (isSelected ?? hasActiveFilters);

A closed filter popover can still have active filters, so this left the trigger exposed to assistive technology as expanded even when the controlled popover was closed.

This changes aria-expanded to follow only the actual disclosure state supplied through isSelected.

Checklist

• PR has been opened against the correct base branch (main)
• Changelog entry added

[github-actions]

👋 Since this is a community submitted pull request, a Buildkite build has not been started automatically. Would an Elastic organization member please verify the contents of this pull request and kick off a build manually?

[weronikaolejniczak]

@DevLikhith5 as mentioned on another PR, 3 things to note: let's limit open PRs, let's focus on issues marked as "help wanted" but because this is genuinely an issue that should likely have that label, I'll leave the PR open. It's hard to say when we get to the review though so I ask for your patience 🙏🏻 Next week our whole team will be unavailable.

[DevLikhith5]

Thanks for the feedback! I'll keep the open PRs limited and focus on "help wanted" issues going forward. No rush on the review, appreciate you keeping it open 🙏

[weronikaolejniczak]

buildkite test this

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: 914632d

History

• 💚 Build #6724 succeeded d24cb6f

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: 914632d
• Documentation website
• Storybook

History

• 💚 Build #3801 succeeded d24cb6f