Bug 51325 - Accessibility: MAS 20B:Repetitive "Change" links displayed with similar name in "options" window
Summary: Accessibility: MAS 20B:Repetitive "Change" links displayed with similar name ...
Status: CLOSED FIXED
Alias: None
Product: Visual Studio Extensions
Classification: Xamarin
Component: Accessibility ()
Version: 4.3.0 (C9)
Hardware: PC Windows
: --- normal
Target Milestone: ---
Assignee: Adrian Murphy
URL:
Depends on:
Blocks:
 
Reported: 2017-01-10 11:48 UTC by v-sukals@microsoft.com
Modified: 2017-09-28 12:03 UTC (History)
4 users (show)

Tags: a11ymas,wipro
Is this bug a regression?: ---
Last known good build:


Attachments
Screenshots of Narrator buddy and inspect (53.15 KB, image/png)
2017-01-10 11:48 UTC, v-sukals@microsoft.com
Details
Inspect Screenshot (99.95 KB, image/png)
2017-01-10 15:05 UTC, v-sukals@microsoft.com
Details
Inspect log_Name property issue for "Browse button" (149.16 KB, image/png)
2017-09-05 09:39 UTC, v-sukals@microsoft.com
Details
Narrator buddy_Name property issue for "Browse" button (143.03 KB, image/png)
2017-09-05 09:40 UTC, v-sukals@microsoft.com
Details
Inspect_Issue is fixed_Name property is proper for Change button (112.24 KB, image/png)
2017-09-28 11:58 UTC, v-sukals@microsoft.com
Details
Narrator buddy_Issue is fixed_Name property is proper for Change button (87.51 KB, image/png)
2017-09-28 11:59 UTC, v-sukals@microsoft.com
Details


Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
CLOSED FIXED

Description v-sukals@microsoft.com 2017-01-10 11:48:42 UTC
Created attachment 19145 [details]
Screenshots of Narrator buddy and inspect

Repro Steps:
1. Launch Visual Studio RC 2017
2. Open File+New Project+Select "Crossplatform" from the Left Pane
3. Select "Cross Platform App"."New Crossplatform" window opens
4. Select "Blank App" 
5. Press Ok button 
6. Open Tools-->Options
7. "Options" window opens. Navigate to "Xamarin" and expand. 
8. Select "Android Settings"
9. Launch Inspect tool
10. Using tab navigate to "change" link
11. Observe "Name" property in inspect

Actual Behavior:
Link Name is defined as "Change" for three different links in the same window


Expected Behavior:
Identical  name should not be defined repetitively in the same window.
i) Name Property for the First "Change" link should be "Change Java Development Kit Location" 
ii) For the second link "Android SDK location Change"
iii)For the third link "Android NDK Location"

User Impact:
Screen Reader will read the link with identical Name for three times which creates ambiguity to visually impaired users
Comment 1 v-sukals@microsoft.com 2017-01-10 15:05:57 UTC
Created attachment 19152 [details]
Inspect Screenshot
Comment 2 xamarin-release-manager 2017-08-29 14:51:34 UTC
Fixed in version 4.8.0.286 (master)

Author: Nick
Commit: 743fbc657abe8d4adcc66ff564e0854c1d4af47d (xamarin/XamarinVS)
Comment 3 v-sukals@microsoft.com 2017-09-05 09:28:48 UTC
UI element has been changed. Instead of "Change" link, "Three dotted buttons" are used in the window, hence closing the bug.

Verified Visual Studio Build:
Visual Studio Enterprise 2017 d15rel
Version 15.4.0 Preview 2.0[26831.0.d15rel]
Comment 4 v-sukals@microsoft.com 2017-09-05 09:39:19 UTC
Created attachment 24564 [details]
Inspect log_Name property issue for "Browse button"
Comment 5 v-sukals@microsoft.com 2017-09-05 09:40:00 UTC
Created attachment 24565 [details]
Narrator buddy_Name property issue for "Browse" button
Comment 6 v-sukals@microsoft.com 2017-09-05 09:53:39 UTC
UI element has been changed. Instead of "Change" link "Browse" button is used in the "Options" window.

Name property is defined as "..." for the buttons which exist in "Options window, hence reactivating the bug.

Name Property should be defined for the buttons as below: 
1.The name for the button beside "Java Development Kit Location" should be "Browse Java Development Kit Location"
2.The name for the button beside "Android SDK Location" should be "Browse Android SDK Location"
2.The name for the button beside "Android NDK Location" should be "Browse Android SDK Location "

Verified Windows Version:
Windows 10 Pro
Version 1703 (OS Build 15063.540)

Verified Visual Studio Build:
Visual Studio Enterprise 2017 d15rel
Version 15.4.0 Preview 2.0[26831.0.d15rel]
Comment 7 Pierce Boggan [MSFT] 2017-09-11 17:51:44 UTC
We have a PR up with to fix this, moving to CONFIRMED until we get that merged:
https://github.com/xamarin/XamarinVS/pull/8122
Comment 8 Nick Sadovyi 2017-09-12 09:11:46 UTC
Fixed (master)

Author: Nick
Commit: 8938b578eae13b1312b420fc0e178ca4dafc065f (xamarin/XamarinVS)
Comment 9 v-sukals@microsoft.com 2017-09-19 05:00:15 UTC
Xamarin version (master) is not available to verify the issue
Comment 10 v-sukals@microsoft.com 2017-09-28 11:58:28 UTC
Created attachment 24986 [details]
Inspect_Issue is fixed_Name property is proper for Change button
Comment 11 v-sukals@microsoft.com 2017-09-28 11:59:20 UTC
Created attachment 24987 [details]
Narrator buddy_Issue is fixed_Name property is proper for Change button
Comment 12 v-sukals@microsoft.com 2017-09-28 12:03:56 UTC
Verified the issue in below mentioned build and the issue got fixed, hence closing the bug.

Verified Windows Version:
Windows 10 Pro
Version 1703 (OS Build 15063.540)

Verified Visual Studio Build:
Visual Studio Enterprise 2017 d15rel
Version 15.5.0 Preview 1.0[26927.3.d15rel]
Xamarin Version: 4.8.0.315(50450332e)

Note: Please find the attached screenshot for more info.