[A11y Assessment - NTP] Issues with interacting with site tiles |
||||||||||||||||||
Issue descriptionChrome Version: OS: Mac What steps will reproduce the problem? (1) Enable Voiceover and use it to navigate through the NTP page, to the tile area (2) Attempt to navigate through the tiles and remove one from the page Two issues: 1) The whole tile section is announced as Frame Zero. This needs to be better and more descriptive. 2) The tiles don't appear to be marked up well for VoiceOver. You can technically get to everything you need to here, but in order to get to the close button, you have to focus the tile with VoiceOver, and then know to then press Ctrl Option Shift Down to interact with the items inside it. We need to figure out a way to tell VoiceOver to tell the user that this is interactive. There is no verbal indication of this right now, so unless I saw the X button visually, I wouldn't know to interact with it.
,
Mar 9 2017
,
Mar 10 2017
Adding a bit more context from testing on Chrome OS. When using ChromeVox to navigate through these, you simply can't get to the X close buttons using ChromeVox or just the keyboard. These X buttons are only available on mouse hover. If we don't want to go as far as to make each of these X buttons accessible with the keyboard, we need to at the very least put the option to remove the tile from the NTP in the context menu.
,
Mar 27 2017
,
Mar 28 2017
,
Apr 21 2017
,
Aug 3 2017
Marking this Available pending the new NTP.
,
Oct 16 2017
Any updates on timing regarding the new NTP?
,
Nov 3 2017
The X buttons to remove tiles are now keyboard-focusable (M64 or later), both on the local and the remote NTP, so I believe that part is fixed. I don't think the iframe has a proper title yet, but that should be easy to fix. Any guidance on what the string should be? "Most visited"?
,
Nov 3 2017
+srahim, +maxwalker
re: c#9 - label for a tile(that's what 'iframe' refers to, correct?)
- how about "{page title}, button"?
- 'button' because it looks and works like a button
- '{page title}' is used for tooltip as well.
- Although a challenge with "{page title}' is when its url is used as title, url seems the next alternative to identify a page when a title was not specified.
,
Nov 3 2017
Sorry, no, the "iframe" is the whole tiles section - what's announced as "Frame Zero" according to the original report. The individual tiles do have the title in an "aria-label" property, so I'd hope they are being announced as something sensible. They are links rather than buttons, which I think makes sense, since they do essentially behave like a link.
,
Nov 3 2017
+ lpalmaro@, leberly@, srahim@, Please advise here more regarding labeling a group element (I'm new to that area). Thanks for clarifying treib@. re: whole titles section - How about "Most visited sites, 8 links" for the grouping(i.e. iframe)? re: title - 'link' sounds reasonable to me as well - adding "x of y" at the end seems useful as well e.g. 'Twitter, link, 1 of 8'
,
Nov 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7bd69c3b82d7faa7060703854ee2930a97d20175 commit 7bd69c3b82d7faa7060703854ee2930a97d20175 Author: Marc Treib <treib@chromium.org> Date: Mon Nov 27 15:53:12 2017 Local NTP: Set a title on the most visited iframe Bug: 777167, 699688 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I6688363fa9c1d996f3d0d6bb1f27fc4720f803df Reviewed-on: https://chromium-review.googlesource.com/789846 Reviewed-by: Friedrich Horschig <fhorschig@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#519309} [modify] https://crrev.com/7bd69c3b82d7faa7060703854ee2930a97d20175/chrome/browser/resources/local_ntp/local_ntp.js [modify] https://crrev.com/7bd69c3b82d7faa7060703854ee2930a97d20175/chrome/browser/search/local_ntp_source.cc
,
Nov 27 2017
The above CL sets a title of "Most visited" for the iframe. The counts would be a nice addition, but they might be hard to translate (and anyway will be more work).
,
Nov 28 2017
I think the worst things here have been addressed: The confusing "frame zero" announcement is gone, and the "x" buttons on the tiles are now tabbable via the keyboard (which I hope also makes them accessible via VoiceOver). Reducing prio for the remaining polish (count and "i of n" announcement).
,
Dec 12 2017
This bug is related to https://bugs.chromium.org/p/chromium/issues/detail?id=696272 This bug is left open for the remaining polish (count and "i of n" announcement). 696272 is for being able to remove tiles using the keyboard
,
Dec 14 2017
,
Dec 15 2017
,
Dec 15 2017
,
Jan 11 2018
Reducing prio since only some polish remains.
,
Jan 11 2018
,
Jul 10
,
Aug 9
Fixed in latest Canary |
||||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||||
Comment 1 by lpalmaro@chromium.org
, Mar 8 2017