diff options
| author | calderwoodra <calderwoodra@google.com> | 2017-11-21 10:54:36 -0800 |
|---|---|---|
| committer | Copybara-Service <copybara-piper@google.com> | 2017-11-21 12:15:19 -0800 |
| commit | e51ca38d99bdc4b5d88b0bacf3905c927c3656bf (patch) | |
| tree | a158b243eac40a00405e58f79d46b3adc3d0a625 /java/com/android/dialer/app | |
| parent | 6d67975290cae16971dd30779ed60dba44962ecc (diff) | |
Reduced jank when placing a call from search UI.
I accidently made the search/dialtacts UI TOO performant and everything was
happening too quickly. Postponed closing the dialpad and exiting search until
onPause, but immediately close the keyboard so it isn't visible over the in
call ui.
Bug: 69313203,68857994
Test: see video
PiperOrigin-RevId: 176531801
Change-Id: I52729c451d46ad00ac87c47f189134694554d33a
Diffstat (limited to 'java/com/android/dialer/app')
| -rw-r--r-- | java/com/android/dialer/app/DialtactsActivity.java | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/java/com/android/dialer/app/DialtactsActivity.java b/java/com/android/dialer/app/DialtactsActivity.java index d9a63fab2..755b335ec 100644 --- a/java/com/android/dialer/app/DialtactsActivity.java +++ b/java/com/android/dialer/app/DialtactsActivity.java @@ -907,8 +907,7 @@ public class DialtactsActivity extends TransactionSafeActivity @Override public void onCallPlacedFromDialpad() { - hideDialpadFragment(false /* animate */, true /*clearDialpad */); - exitSearchUi(); + mClearSearchOnPause = true; } @Override @@ -1021,9 +1020,8 @@ public class DialtactsActivity extends TransactionSafeActivity private void hideDialpadAndSearchUi() { if (mIsDialpadShown) { hideDialpadFragment(false, true); - } else { - exitSearchUi(); } + exitSearchUi(); } private void prepareVoiceSearchButton() { @@ -1678,11 +1676,9 @@ public class DialtactsActivity extends TransactionSafeActivity } @Override - public void onCallPlaced() { - if (mIsDialpadShown) { - hideDialpadFragment(false, true); - } - exitSearchUi(); + public void onCallPlacedFromSearch() { + DialerUtils.hideInputMethod(mParentLayout); + mClearSearchOnPause = true; } protected int getPreviouslySelectedTabIndex() { |