search

USDepartmentofLabor / Local-Labor-Market-Data-Android

Android version of BLS Local Labor Market Data
0 stars 0 forks source link

Accessibility - Metro - The Word "City Name" Needs to be a Heading #7

Closed jfc3-dol closed 5 years ago

jfc3-dol commented 5 years ago

On the county screen the "City Name" is not set as a heading. Please make the "City Name" a heading so TalkBack users know what screen they are on.

greg-gruse commented 5 years ago

This will be fixed in the next build.

greg-gruse commented 5 years ago

This should be fixed in the May 8th build?

jfc3-dol commented 5 years ago

Thanks, for making the "City Name" now a heading but why is the text in the gray boxes not a heading anymore? Thought that text such as "Unemployment Rate", "Industry Employment", etc. were headings before and are not now? TalkBack is reading it as a list item for some reason.

greg-gruse commented 5 years ago

Sorry to say this again but the gray boxes are reading as headings on my device (which is a Pixel 2 - Android version 9), let me know your os and device and I will investigate.

jfc3-dol commented 5 years ago

We are using two devices for are testing and they are the following:

greg-gruse commented 5 years ago

Fixed in the next build

jfc3-dol commented 5 years ago

The text in the gray boxes which are supposed to be headings are not being read as a heading. Some even read them as text with a list of "X" number of items.

jfc3-dol commented 5 years ago

TalkBack is reading the text in the gray boxes as a heading now.

So I'm going to close the issue.