Skip to content

button name move assumption to the correct section#2286

Merged
WilcoFiers merged 3 commits intodevelopfrom
WilcoFiers-patch-3
May 8, 2025
Merged

button name move assumption to the correct section#2286
WilcoFiers merged 3 commits intodevelopfrom
WilcoFiers-patch-3

Conversation

@WilcoFiers
Copy link
Copy Markdown
Member

This is an assumption, not an accessibility support issue.

Need for Call for Review: none


How to Review And Approve

  • Go to the “Files changed” tab
  • Here you will have the option to leave comments on different lines.
  • Once the review is completed, find the “Review changes” button in the top right, select “Approve” (if you are really confident in the rule) or "Request changes" and click “Submit review”.
  • Make sure to also review the proposed Call for Review period. In case of disagreement, the longer period wins.


- The rule assumes that all buttons are [user interface components as defined by WCAG 2](https://www.w3.org/TR/WCAG22/#dfn-user-interface-components).

- Some elements have a role of `button` and a default accessible name defined by the [HTML Accessibility API Mapping][html aam input button], for example `input` elements whose `type` [attribute value][] is either `submit` or `reset`. This rule considers that these default names can be descriptive and therefore does not fail them.
Copy link
Copy Markdown
Collaborator

@zlayaAvocado zlayaAvocado May 1, 2025

Choose a reason for hiding this comment

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

Should the HTML Accessibility API Mapping link be updated? It seems it's currently not referencing anything and goes to the whole file instead, new link: https://www.w3.org/TR/html-aam-1.0/#input-type-button-input-type-submit-and-input-type-reset-accessible-name-computation

The same issue is in the "Bibliography" section of the rule

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch

@netlify
Copy link
Copy Markdown

netlify Bot commented May 2, 2025

Deploy Preview for act-rules failed.

Name Link
🔨 Latest commit 8a678cc
🔍 Latest deploy log https://app.netlify.com/sites/act-rules/deploys/68149a31dfdfbe000827c574

@WilcoFiers WilcoFiers requested a review from zlayaAvocado May 2, 2025 10:11
@WilcoFiers WilcoFiers merged commit b0d82c7 into develop May 8, 2025
2 of 6 checks passed
@WilcoFiers WilcoFiers deleted the WilcoFiers-patch-3 branch May 8, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants