search

microsoft / moaw

Grab-and-go resources to help you learn new skills, with all the tools you need to create, host and share your own workshop
https://aka.ms/moaw
Creative Commons Attribution Share Alike 4.0 International
103 stars 77 forks source link

Multiple H1 heading is defined on the home page: A11y_Maow_Home Page_Info and Relationships. #109

Closed AnKushSingh05 closed 4 months ago

AnKushSingh05 commented 5 months ago

GitHub Tags

A11yTCS; #A11ySev2; #ChromiumEdge; #WCAG1.3.1; #Win11; #GH_Maow_Web_Mar24; #Maow; #Info and Relationships; #AILimited; #DesktopWeb; #A11yMAS;#Closed;

Actual:

Multiple H1 heading is defined on the home page.

Expected:

Only one H1 heading should be defined on the page.

Environment Details:

Application Name: Maow

URL: https://moaw.dev/

OS: Windows 11 Enterprise 23H2 (OS build 22631.3296) Microsoft Edge: Version 122.0.2365.92 (Official build) (64-bit)

Repro Steps:

  1. Hit the URL https://moaw.dev/
  2. Moaw screen will get open.
  3. TAB till header section.
  4. Verify that Multiple H1 heading is defined on the home page or not.

    User Impact:

    User not able to understand the current structure of the page if multiple H1 heading level tag present on the same page.

Attachment:

Presentation2

sinedied commented 4 months ago

@AnKushSingh05 this one has been fixed, could you validate it?

AnKushSingh05 commented 3 months ago

Verified the issue in below environment https://moaw.dev/ Issue is fixed. Hence closing the bug

https://github.com/microsoft/moaw/assets/124244478/6ec6526f-b64a-4125-ad76-158ffce2a6db

sinedied commented 3 months ago

:tada: This issue has been resolved in version 1.5.0 :tada:

The release is available on npm package (@latest dist-tag)

Your semantic-release bot :package::rocket: