Issue metadata
Sign in to add a comment
|
jsbin output accessible bounding boxes are wrong |
||||||||||||||||||||||
Issue descriptionTo repro: 1. Visit jsbin.com 2. Create an html page with a focusable element like an <input> 3. Tab to the text field in the output 4. Enable ChromeVox, VoiceOver, or another screen reader that shows bounding boxes. Note that it's offset. The underlying issue is that the iframe element has padding.
,
Nov 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19 commit ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19 Author: Dominic Mazzoni <dmazzoni@chromium.org> Date: Wed Nov 22 23:17:18 2017 Iframe accessible bounding boxes should include border,padding. We store accessible bounding boxes as relative coordinates, and normally each element takes its parent's border and padding into account when exposing its own top/left coordinate. This doesn't work for the root document node inside an iframe, because it can't know the owning iframe element's border and padding, so we need a special case for that. Bug: 740312 Change-Id: I4c7d7f435eff9c69afa889d666bd09a2dc5716ae Reviewed-on: https://chromium-review.googlesource.com/775024 Reviewed-by: Aaron Leventhal <aleventhal@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#518794} [modify] https://crrev.com/ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19/content/browser/accessibility/dump_accessibility_tree_browsertest.cc [add] https://crrev.com/ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19/content/test/data/accessibility/html/iframe-padding-expected-blink.txt [add] https://crrev.com/ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19/content/test/data/accessibility/html/iframe-padding.html [modify] https://crrev.com/ca8e25b9ac0ded66124c5d771cc4c8d4157b0d19/third_party/WebKit/Source/modules/accessibility/AXObject.cpp
,
Dec 12 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dmazz...@chromium.org
, Nov 16 2017