Skip to content

[$40]Links must have discernible text #3097

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

Closed
kkartunov opened this issue Jul 25, 2019 · 15 comments
Closed

[$40]Links must have discernible text #3097

kkartunov opened this issue Jul 25, 2019 · 15 comments
Assignees

Comments

@kkartunov
Copy link
Collaborator

kkartunov commented Jul 25, 2019

Steps to Reproduce

  1. Go to https://www.topcoder.com/community/accessibility-testing
  2. Open DevTools (right click -> inspect)
  3. In the Elements tab, look at the element: <a href="https://www.topcoder.com" class="_2WLJ0i">

Expected Result
Links should have discernible text

Actual Result
Element link doesn’t have discernible text. Link text might not be accessible to screen readers

Device: Desktop/Labtop
Browser: Chrome

WCAG Category: 2. Operable
WCAG Level: A
WCAG Success Criterion: 2.4.4 Link Purpose (In Context)
Screen Reader Used: N/A
Accessibility Audit Tool used: N/A

Attachment
https://topcodermsft-my.sharepoint.com/personal/msft-copilots_topcodermsft_onmicrosoft_com/Documents/Forms/All.aspx?cid=2521ab22%2D4cea%2D4606%2Da210%2D50280914399c&FolderCTID=0x0120005C598A51049FC14CBA882E1AEE168F51&id=%2Fpersonal%2Fmsft%2Dcopilots%5Ftopcodermsft%5Fonmicrosoft%5Fcom%2FDocuments%2F%5BFAST%5D%20Test%20for%20Accessibility%20Series%2Fyoctocode%2Fbug%206

@kkartunov kkartunov changed the title [FAST-11] Links must have discernible text [$40]Links must have discernible text Jul 26, 2019
@kkartunov
Copy link
Collaborator Author

@crazyk07
Copy link

Contest https://www.topcoder.com/challenges/30097737 has been created for this ticket.

This is an automated message for crazyk via Topcoder X

@crazyk07
Copy link

Contest https://www.topcoder.com/challenges/30097737 has been updated - the new changes has been updated for this ticket.

This is an automated message for crazyk via Topcoder X

@kkartunov
Copy link
Collaborator Author

@topcoder-platform/topcodercompetitors this open for pickup

@romitgit
Copy link

Access denied for the linked files.

@crazyk07
Copy link

Contest https://www.topcoder.com/challenges/30097737 has been updated - it has been assigned to rohitgupta_.

This is an automated message for crazyk via Topcoder X

@r0hit-gupta
Copy link
Collaborator

Already fixed by #2970

@crazyk07
Copy link

Contest https://www.topcoder.com/challenges/30097737 has been updated - the new changes has been updated for this ticket.

This is an automated message for crazyk via Topcoder X

@kkartunov
Copy link
Collaborator Author

No @r0hit-gupta . This ticket is for all links on that page! Please, update or reopen it for pickup if you won't.

@r0hit-gupta
Copy link
Collaborator

Steps to Reproduce

  1. Go to https://www.topcoder.com/community/accessibility-testing
  2. Open DevTools (right click -> inspect)
  3. In the Elements tab, look at the element: <a href="https://www.topcoder.com" class="_2WLJ0i">

Expected Result
Links should have discernible text

Actual Result
Element link doesn’t have discernible text. Link text might not be accessible to screen readers

Device: Desktop/Labtop
Browser: Chrome

WCAG Category: 2. Operable
WCAG Level: A
WCAG Success Criterion: 2.4.4 Link Purpose (In Context)
Screen Reader Used: N/A
Accessibility Audit Tool used: N/A

Attachment
https://topcodermsft-my.sharepoint.com/personal/msft-copilots_topcodermsft_onmicrosoft_com/Documents/Forms/All.aspx?cid=2521ab22%2D4cea%2D4606%2Da210%2D50280914399c&FolderCTID=0x0120005C598A51049FC14CBA882E1AEE168F51&id=%2Fpersonal%2Fmsft%2Dcopilots%5Ftopcodermsft%5Fonmicrosoft%5Fcom%2FDocuments%2F%5BFAST%5D%20Test%20for%20Accessibility%20Series%2Fyoctocode%2Fbug%206

@kkartunov both the attachment screenshots and steps to reproduce describe only a single link with the issue. I could not find other links with the same issue. Please review

@kkartunov
Copy link
Collaborator Author

@r0hit-gupta those in the footer need to be handled too.

@r0hit-gupta
Copy link
Collaborator

@kkartunov we don't need to add labels to the footer links. They already have text that is visible to screen readers.

@kkartunov
Copy link
Collaborator Author

@r0hit-gupta you are right. Sorry, but have to mark this as duplicate and cancel it. Thanks for your feedback!

@crazyk07
Copy link

This ticket was not processed for payment. If you would like to process it for payment, please reopen it, add the tcx_FixAccepted label, and then close it again

This is an automated message for crazyk via Topcoder X

@r0hit-gupta
Copy link
Collaborator

@veshu @kkartunov will I not be paid for this?

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

No branches or pull requests

5 participants