Commit 81c5f7e3 authored by Winson's avatar Winson Committed by Winson Chung
Browse files

Fixing issue with search bar not being visible.

- If you queue up two SearchBarDropTarget state transitions one right after
  another, the second animation will skip (since the values haven't started
  animating yet) and the first animation will complete instead, leaving the
  visibility of the search and drop target bars at odds with the current
  bar state.
- Instead, we should cancel all existing animators first, such that if we
  are already in the final state, no animations will run, and the correct
  bars will be visibile.

Bug: 23201830
Change-Id: I1f8e802821ef2b4904a3efa7edbea6ae615479b9
parent 6779595b
......@@ -186,7 +186,12 @@ public class SearchDropTargetBar extends FrameLayout implements DragController.D
*/
private void animateViewAlpha(LauncherViewPropertyAnimator animator, View v, float alpha,
int duration) {
if (v != null && Float.compare(v.getAlpha(), alpha) != 0) {
if (v == null) {
return;
}
animator.cancel();
if (Float.compare(v.getAlpha(), alpha) != 0) {
if (duration > 0) {
animator.alpha(alpha).withLayer().setDuration(duration).start();
} else {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment