diff options
author | Svet Ganov <svetoslavganov@google.com> | 2014-12-05 13:02:24 -0800 |
---|---|---|
committer | Svet Ganov <svetoslavganov@google.com> | 2014-12-05 13:13:15 -0800 |
commit | b3fa2787eabd2be6d7780e215db0d9a5904ba47c (patch) | |
tree | 37f44d40812d0f666f5ab6aa9c1121a2753ca1f4 /core/java/android/view/ViewGroup.java | |
parent | 54daab1009e7be849fa1dcbf4ee3f9d0ba40d732 (diff) | |
download | frameworks_base-b3fa2787eabd2be6d7780e215db0d9a5904ba47c.zip frameworks_base-b3fa2787eabd2be6d7780e215db0d9a5904ba47c.tar.gz frameworks_base-b3fa2787eabd2be6d7780e215db0d9a5904ba47c.tar.bz2 |
When accessibility on cannot click on a view covered by a HorizontalScrollView
In accessibility mode we send down and up events activate a view. We will later
switch to accessibility actions but for now as a bridge-gap we compute a point on
the screen where to click for activating the view. The heuristic we use has edge
cases such as a view that handles all touch events but does not have any listeners.
In this case we do not ignore the target view's area covered by a view that handles
all touch events. As a result we click on the wrong target. While we cannot solve
this generically, in the case of standard components such as HorizontalScrollView
we can.
bug:18612258
Change-Id: If8482aac0d0ea53c5c90367d099d1b8d3a4559ed
Diffstat (limited to 'core/java/android/view/ViewGroup.java')
-rw-r--r-- | core/java/android/view/ViewGroup.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/core/java/android/view/ViewGroup.java b/core/java/android/view/ViewGroup.java index 1551504..64c8ca6 100644 --- a/core/java/android/view/ViewGroup.java +++ b/core/java/android/view/ViewGroup.java @@ -880,8 +880,11 @@ public abstract class ViewGroup extends View implements ViewParent, ViewManager return true; } + /** + * @hide + */ @Override - void addClickableRectsForAccessibility(List<RectF> outRects) { + public void addClickableRectsForAccessibility(List<RectF> outRects) { int sizeBefore = outRects.size(); super.addClickableRectsForAccessibility(outRects); |