-
Notifications
You must be signed in to change notification settings - Fork 212
[$40][MSFT-122] Sample challenge page: Improper heading level structure #2820
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
Comments
Contest https://www.topcoder.com/challenges/30096003 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30096003 has been updated - the new changes has been updated for this ticket. |
@veshu I don't understand what is the problem in this, if we try to change something, it will break the design. What is the purpose of doing it in proper structure? |
Contest https://www.topcoder.com/challenges/30096003 has been updated - the new changes has been updated for this ticket. |
@asadath1395 I will get back to you on this later. please continue with other issues for now. Thanks! |
@drasticdpk Can you please share input here #2820 (comment) |
Hi @veshu / @asadath1395 , currently the header is not on structure way in give page. Here is the reference to update header in correct way. |
Contest https://www.topcoder.com/challenges/30097553 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30097553 has been updated - it has been assigned to Timor. |
PR for this issue is ready : #3056 |
Operating System: Windows |
@veshu , Since some header are not showing in Test env due to configuration/Setup dependency. It would be great if we could setup all scenario in test environment , so it has less chance to miss all scenario by developer. |
Should I commit a fix for this @drasticdpk ? |
@nkumar-topcoder @sushilshinde - this is one of the Env dependent issues as raised in the call yesterday. Unable to test it completely in Dev. Heading structure as visible in Dev was tested and passed by @drasticdpk. Hence failed in Prod. @drasticdpk - could you add image of this page on Dev to clarify? |
@drasticdpk there was a patch provided Please test after 15 mins on test env |
Verified in prod. |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30097553 |
Steps to Reproduce
Expected Result
Use h1-h6 tags in a logical sequence. Headings should not skip levels.
Actual Result
Improper heading level structure. The first heading 'the name of the challenge' on the page is a level 1 heading followed by 'prizes' at level 3. 'eligible events', 'review style', 'challenge links', 'challenge terms', 'share' also need to be designated with the correct heading levels.
Device: Desktop/Labtop
Operating System: Windows
Browser: Chrome
WCAG Category: 1. Perceivable
WCAG Level: A
WCAG Success Criterion: 1.3.1 Info and Relationships
Screen Reader Used: JAWS
Accessibility Audit Tool used: N/A
Attachment
https://topcodermsft-my.sharepoint.com/personal/pd-topcoder_topcodermsft_onmicrosoft_com/Documents/Forms/All.aspx?cid=088f5f5d%2D9b1c%2D4b7e%2Dacbf%2D320c257c9a9a&FolderCTID=0x0120005C598A51049FC14CBA882E1AEE168F51&id=%2Fpersonal%2Fpd%2Dtopcoder%5Ftopcodermsft%5Fonmicrosoft%5Fcom%2FDocuments%2FMSFT%2DTeams%2DQA%2FJune%202019%2F29%20Jun%2FTest%20Results%2FBug%20Videos%20%26%20Log%20Files%2FKameshwarid%2Fbug%2020
The text was updated successfully, but these errors were encountered: