Skip to content

[$40][MSFT-240] [Register][NVD][Edge] Input default text read-out twice by the screen-reader #2895

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
veshu opened this issue Jul 17, 2019 · 32 comments

Comments

@veshu
Copy link
Contributor

veshu commented Jul 17, 2019

Steps to Reproduce

  1. Go to https://accounts.topcoder.com/member
  2. Press Tab button to go through the form

Expected Result
Should read-out only one time

Actual Result
Input default text read-out twice by the screen-reader

Device: Desktop/Labtop
Operating System: Windows
Browser: Edge

WCAG Category: 1. Perceivable
WCAG Level: A
WCAG Success Criterion: 1.3.1 Info and Relationships
Screen Reader Used: Other
Accessibility Audit Tool used: N/A

Attachment
https://topcodermsft-my.sharepoint.com/personal/pd-topcoder_topcodermsft_onmicrosoft_com/Documents/Forms/All.aspx?cid=58da9b68%2Df2a3%2D45d2%2Dbbae%2D15abdfd0865b&FolderCTID=0x0120005C598A51049FC14CBA882E1AEE168F51&sortField=Modified&isAscending=false&id=%2Fpersonal%2Fpd%2Dtopcoder%5Ftopcodermsft%5Fonmicrosoft%5Fcom%2FDocuments%2FMSFT%2DTeams%2DQA%2FJune%202019%2F29%20Jun%2FTest%20Results%2FBug%20Videos%20%26%20Log%20Files%2Fcodejam%2DAccessibility%2F74

@crazyk07 crazyk07 added the Login label Jul 17, 2019
@mishacucicea mishacucicea changed the title [MSFT-240] [Register][NVD][Edge] Input default text read-out twice by the screen-reader [$20][MSFT-240] [Register][NVD][Edge] Input default text read-out twice by the screen-reader Jul 18, 2019
@cwdcwd
Copy link
Contributor

cwdcwd commented Jul 18, 2019

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

This is an automated message for lazybaer via Topcoder X

@cwdcwd
Copy link
Contributor

cwdcwd commented Jul 18, 2019

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

This is an automated message for lazybaer via Topcoder X

@Nachiappa
Copy link

I tried to reproduce this issue , but couldn't . Here is my recording

Topcoder - Google Chrome 2019-07-23 08-20-22.zip

@veshu veshu changed the title [$20][MSFT-240] [Register][NVD][Edge] Input default text read-out twice by the screen-reader [$40][MSFT-240] [Register][NVD][Edge] Input default text read-out twice by the screen-reader Jul 23, 2019
@cwdcwd
Copy link
Contributor

cwdcwd commented Jul 23, 2019

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

This is an automated message for lazybaer via Topcoder X

@crazyk07
Copy link

Contest https://www.topcoder.com/challenges/30097858 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/30097858 has been updated - it has been assigned to rashmi73.

This is an automated message for crazyk via Topcoder X

@crazyk07
Copy link

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

This is an automated message for crazyk via Topcoder X

@r0hit-gupta
Copy link
Collaborator

@veshu @codeMinter unable to reproduce. Should have been fixed by other PRs probably.

Test URL - https://accounts.topcoder.com/member
Env - Windows/Edge
Screen Reader Used - NVDA

@r0hit-gupta
Copy link
Collaborator

@nkumar-topcoder @tosha5252 everything is working fine for me locally on branch qa-accessibility.

Win/Edge/NVDA

@nkumar-topcoder
Copy link
Contributor

@tosha5252 I have merged to dev. can you pls check.

@tosha5252
Copy link

Announcement of fields twice is still happening
Win10/Edge/NVDA

Pls see video
TinyTake by MangoApps-19-08-2019-10-57-56.zip

@tosha5252 tosha5252 added Can't Reproduce QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed Ready for QA Can't Reproduce labels Aug 20, 2019
@nkumar-topcoder
Copy link
Contributor

nkumar-topcoder commented Aug 20, 2019

@r0hit-gupta can you please check. Earlier too it had issue on qa-accessibility branch post merge to dev same status.

If you are plan to fix, take the affected files from dev branch.

@r0hit-gupta
Copy link
Collaborator

@nkumar-topcoder @tosha5252 I am really sorry but I had fixed only the login page. The issue body mentioned the URL https://accounts.topcoder.com/member so I did not change the registration form. Will fix it now. @tosha5252 thanks for the video reference.

@nkumar-topcoder
Copy link
Contributor

@nkumar-topcoder @tosha5252 I am really sorry but I had fixed only the login page. The issue body mentioned the URL https://accounts.topcoder.com/member so I did not change the registration form. Will fix it now. @tosha5252 thanks for the video reference.

thanks @tosha5252

@nkumar-topcoder nkumar-topcoder added tcx_Feedback and removed QA Fail QA verification on Dev has failed. Assignee to redo the fix. labels Aug 21, 2019
@r0hit-gupta
Copy link
Collaborator

@nkumar-topcoder @tosha5252 @mishacucicea created a new PR. This should properly fix both the registration page and the login page. Please review.

@crazyk07
Copy link

Closing ticket due to shift in priority, will not be able to complete but will still pay for effort rendered.

@crazyk07
Copy link

Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30097858

This is an automated message for crazyk via Topcoder X

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests