Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Put label-same-as-description (from label / select-name) into its own rule #1325

Closed
Tracked by #4066
WilcoFiers opened this issue Jan 17, 2019 · 8 comments · Fixed by #4096
Closed
Tracked by #4066

Put label-same-as-description (from label / select-name) into its own rule #1325

WilcoFiers opened this issue Jan 17, 2019 · 8 comments · Fixed by #4096
Assignees
Labels
feat New feature or enhancement rules Issue or false result from an axe-core rule
Milestone

Comments

@WilcoFiers
Copy link
Contributor

The help-same-as-label check is the only check in axe-core disabled by default. This is confusing. We should either put it in its own rule, have it run as a best practice, or we should remove it.

@WilcoFiers WilcoFiers added feat New feature or enhancement rules Issue or false result from an axe-core rule labels Jan 17, 2019
@dylanb
Copy link
Contributor

dylanb commented Jun 16, 2019

Should be a best practice rule

@straker straker added this to the Axe-core 4.0 milestone Nov 25, 2019
@WilcoFiers WilcoFiers modified the milestones: Axe-core 4.0, axe-core 4.1 Apr 28, 2020
@WilcoFiers WilcoFiers modified the milestones: axe-core 4.1, axe-core 4.2 Sep 9, 2020
@WilcoFiers WilcoFiers modified the milestones: axe-core 4.2, axe-core 4.3 Jan 7, 2021
@WilcoFiers WilcoFiers modified the milestones: axe-core 4.3, Axe-core 4.4 May 12, 2021
@WilcoFiers WilcoFiers modified the milestones: Axe-core 4.4, Axe-core 4.5 Nov 10, 2021
@WilcoFiers WilcoFiers modified the milestones: Axe-core 4.5, Axe-core 4.6 May 19, 2022
@WilcoFiers WilcoFiers modified the milestones: Axe-core 4.6, Axe-core 4.7 Nov 2, 2022
@WilcoFiers WilcoFiers changed the title Put help-same-as-label into its own rule Put label-same-as-description (from label / select-name) into its own rule Jun 28, 2023
@WilcoFiers
Copy link
Contributor Author

WilcoFiers commented Jun 28, 2023

Looking at this some more. I don't think the current check is suitable to be put into its own rule. Doing a simple strA !== strB isn't particularly insightful. I don't think I can justify working to create a new best practice, given the other priorities we have. I don't think this check should be deprecated. It has some use for custom rules, but it's not suitable to run by default.

Instead, I'm going to enable this check for anyone who wants it in a custom rule, and I'm going to remove it from the label and select-name rules so that this check doesn't run by default, and its impact doesn't get listed in the rule-descriptions.md file.

@padmavemulapati
Copy link

Valdiated using latest axe-core develop branch code base,

'label' and 'select-name' rules - now having impact only critical there is no minor impact' .

but not seeing any new rules listed in rules with label-same-as-description

Image

@padmavemulapati
Copy link

@straker , can you please help me to get the html snippet to raise an issue for the rule label-same-as-description

@straker
Copy link
Contributor

straker commented Aug 3, 2023

@padmavemulapati You'll need to create a custom rule to use it since we don't have any rules that use it by default anymore.

<input title="Username" aria-label="Select Username">
<script>
axe.configure({
  rules: [
    {
      id: 'help-same-as-label',
      any: ['help-same-as-label']
    }
  ]
});

await axe.run()
</script>

@padmavemulapati
Copy link

@straker , thank you , it is working with custom-rule configuration for help-same-as-label rule.

Screen.Recording.2023-08-03.at.8.10.40.PM.mov

@somaalapati
Copy link

@padmavemulapati Please add environment details in the closing comments.

@padmavemulapati
Copy link

@padmavemulapati Please add environment details in the closing comments.

@somaalapati , axe-core tickets validating on the latest develop branch code base, and adding the axe.js file in source - snippets running the source to axe.run() so obviously it is testing in chrome browser - developer tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or enhancement rules Issue or false result from an axe-core rule
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants