Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Expandables Added aria-controls and aria-expanded #820

Merged
merged 1 commit into from
Apr 7, 2020

Conversation

tomheller
Copy link
Collaborator

Pull Request

Fixed an issue where the correct aria attributes where not present on accordion like elements like
expandable-section, expandable-text and expandable-panel.

Fixes #788

Type of PR

Bugfix

Checklist

  • I have read the CONTRIBUTING doc and I follow the PR guidelines
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@tomheller tomheller self-assigned this Apr 2, 2020
@github-actions
Copy link

github-actions bot commented Apr 2, 2020

Deploy preview for website ready!

Built with commit f97ed92

https://browser-mx1u6hjyr.now.sh

Copy link
Collaborator

@ffriedl89 ffriedl89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding those for increased accessibility. I think we need a little addition for the id, but then we should be fine.

…a-controls and aria-expanded.

Fixed an issue where the correct aria attributes where not present on accordion like elements like
expandable-section, expandable-text and expandable-panel.

Fixes #788
@sonarcloud
Copy link

sonarcloud bot commented Apr 6, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

}

/** Mixin to augment a directive with a `id` property. */
export function mixinId<T extends Constructor<{}>>(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@ffriedl89 ffriedl89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done 💯 more a11y is always good

@ffriedl89 ffriedl89 added the pr: merge-ready This PR is ready to be merged label Apr 7, 2020
@github-actions github-actions bot added target: minor This PR is targeted for the next minor release target: patch This PR is targeted for the next patch release labels Apr 7, 2020
@tomheller tomheller merged commit 4da4ec3 into master Apr 7, 2020
@tomheller tomheller deleted the feat/add-aria-to-expanded branch April 7, 2020 10:46
@github-actions
Copy link

github-actions bot commented Apr 7, 2020

The auto cherry picker succeeded.Cherry picked to minor branch 6.x. New head is now f37a977
Cherry picked to patch branch 6.1.x. New head is now b9f9978

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge-ready This PR is ready to be merged target: minor This PR is targeted for the next minor release target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A11y: Expandables (section, panel, trigger) lack aria properties
3 participants