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

Ensures lang attributes have valid values (pre[lang="bash"]) #21638

Closed
Govardhani-T opened this issue Aug 6, 2024 · 4 comments
Closed

Ensures lang attributes have valid values (pre[lang="bash"]) #21638

Govardhani-T opened this issue Aug 6, 2024 · 4 comments
Assignees
Labels
documentation improvements or additions to documentation; typically submitted using template

Comments

@Govardhani-T
Copy link

Govardhani-T commented Aug 6, 2024

GitHub Tags:

#A11ySev2; #A11yTCS; #DesktopWeb; #SH_ONNX Runtime & Ecosystem_Web_Mar2024; #ONNX Runtime & Ecosystem; #WCP; #Win11; #FTP; #P3_WCP; #ONNX Runtime; #ChromiumEdge; A11yWCAG2.1; #A11yMAS; #valid-lang; #WCAG3.1.2; #A11yAuto #language of parts;
"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver".

Environment Details:

Application Name: WCP URLs - ONNX Runtime
URL: https://onnxruntime.ai/
Edge version: 123.0.2420.53 (Official build) (64-bit)
OS: Windows 11 version 23H2

Steps to Reproduce

  1. Hit the URL "https://onnxruntime.ai/docs/install/".
  2. "ONNX Runtime" AMD ROC-m page appeared.
  3. Run Accessibility Insights for web.
  4. Observe that Ensures lang attributes have valid values (pre[lang="bash"])

Issue:

Ensures lang attributes have valid values (valid-lang - https://accessibilityinsights.io/info-examples/web/valid-lang)

Target application:

Install ONNX Runtime | onnxruntime - https://onnxruntime.ai/docs/install/

Element path:

tr:nth-child(2) > td:nth-child(4) > pre[lang="bash"]

Snippet:

dotnet add package Microsoft.ML.OnnxRuntime.Training

How to fix:

Fix all of the following:
Value of lang attribute not included in the list of valid languages

Environment: Microsoft Edge version 127.0.0.0

====

This accessibility issue was found using Accessibility Insights for Web 2.44.0 (axe-core 4.9.1), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights.

Attachment:

image

@github-actions github-actions bot added the .NET Pull requests that update .net code label Aug 6, 2024
@Samhitha66
Copy link

#Rev:samhithaks;

@sophies927 sophies927 added documentation improvements or additions to documentation; typically submitted using template and removed .NET Pull requests that update .net code labels Aug 8, 2024
@v-jagansai
Copy link

Verified the issue in the environment.:https://onnxruntime.ai/docs/install/
issue is not fixed.PFA
Image

MaanavD added a commit that referenced this issue Nov 14, 2024
Addresses #21638

Changed install to genai, moved genai section up, accessibility changes
in docs/install.

Still todo (another PR, soon to come):

- Change theme for accessibility
- Change videos linked on homepage
- Update homepage content to more closely reflect current ORT status.

Preview at : https://maanavd.github.io/onnxruntime/
@Govardhani-T
Copy link
Author

Verified the issue in the environment. Issue is not reproing. hence closing the bug. PFA for reference.Image

@Govardhani-T
Copy link
Author

#Closed;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation improvements or additions to documentation; typically submitted using template
Projects
None yet
Development

No branches or pull requests

5 participants