Skip to content
Commit 5fd69c38 authored by Evan Rosky's avatar Evan Rosky
Browse files

Fixed poor behavior of position-based focus order

Rect-based focus order algorithm used to sort all views such-that
higher tops were higher. The result is that focus order could
easily move end-to-start when elements with varying heights (even
1px different) were arranged horizontally.

This changes the algorithm to take vertical overlap into account
when sorting. It tries its best to match UX for how focus-order
should work in ambiguous cases.

Bug: 33848452
Bug: 34854951
Test: Checked in a sample-app to make sure focus order matches UX
      more closely than before. Also verified that existing
      FocusFinder CTS tests still pass.

Change-Id: I0e0f2f63d5151693a2642604f64df985856c8991
parent d133b00e
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment