New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment
link

Issue 910258: Improve voiceover reading of search settings recently visited

Reported by ghendel@chromium.org, Nov 29 Project Member

Issue description

When using Voiceover on the search settings page in Canary, the "recently visited" section is verbalized as a button and a heading, but it should only be verbalized as a heading as it's not interactive.
 

Comment 1 by mrsuyi@chromium.org, Dec 14

Status: Started (was: Assigned)

Comment 2 by bugdroid1@chromium.org, Dec 14

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d25d640f7f2ce6bae0a0c7f5711e0af79e622791

commit d25d640f7f2ce6bae0a0c7f5711e0af79e622791
Author: Yi Su <mrsuyi@chromium.org>
Date: Fri Dec 14 18:44:08 2018

Fix a11y traits of TableViewTextHeaderFooterView.

TableViewTextHeaderFooterView has UIAccessibilityTraits of
UIAccessibilityTraitButton and UIAccessibilityTraitHeader, but it's
never used as a button. This CL removes the UIAccessibilityTraitButton.

Bug:  910258 
Change-Id: Id04c72449e5e2cfa3ea85efe3b9ea2cd05c6e2b6
Reviewed-on: https://chromium-review.googlesource.com/c/1378131
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Commit-Queue: Yi Su <mrsuyi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616761}
[modify] https://crrev.com/d25d640f7f2ce6bae0a0c7f5711e0af79e622791/ios/chrome/browser/ui/table_view/cells/table_view_text_header_footer_item.mm

Comment 3 by mrsuyi@chromium.org, Dec 14

Status: Fixed (was: Started)

Comment 4 by yangulo@chromium.org, Dec 18

Status: Verified (was: Fixed)
Issue verified 
Version: Chrome Canary 73.0.3644.0
Device: iPhone XS
iOS: 12.1

Correct voiceover text was read "Recent tabs" is verbalized as only heading

Sign in to add a comment