Skip to content

Remove 'searchbox' from label in name background - #2448

Open
WilcoFiers wants to merge 1 commit into
developfrom
WilcoFiers-patch-3
Open

Remove 'searchbox' from label in name background#2448
WilcoFiers wants to merge 1 commit into
developfrom
WilcoFiers-patch-3

Conversation

@WilcoFiers

Copy link
Copy Markdown
Member

Searchbox is not named from content as the section says.

Need for Call for Review: none, PR is editorial.


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.

@netlify

netlify Bot commented Sep 2, 2026

Copy link
Copy Markdown

Deploy Preview for act-rules ready!

Name Link
🔨 Latest commit 2aacd4e
🔍 Latest deploy log https://app.netlify.com/projects/act-rules/deploys/6a985be84d15000008e785af
😎 Deploy Preview https://deploy-preview-2448--act-rules.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@shunguoy

shunguoy commented Sep 3, 2026 via email

Copy link
Copy Markdown
Collaborator

@shunguoy
shunguoy requested review from shunguoy and removed request for shunguoy September 4, 2026 13:51
## Background

This rule applies to elements with a [widget role][] that [support name from content][supports name from content]. This includes the following: `button`, `checkbox`, `gridcell`, `link`, `menuitem`, `menuitemcheckbox`, `menuitemradio`, `option`, `radio`, `searchbox`, `switch`, `tab`, `treeitem`.
This rule applies to elements with a [widget role][] that [support name from content][supports name from content]. This includes the following: `button`, `checkbox`, `gridcell`, `link`, `menuitem`, `menuitemcheckbox`, `menuitemradio`, `option`, `radio`, `switch`, `tab`, `treeitem`.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just wondering if this rule should be expanded to include all the labelable elements ? If a labelable element takes a role which has an author-supplied accessible name other than from label itself, then same requirement is applied. It's fine for a new issue, rather than included in the current one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants