diff options
author | calderwoodra <calderwoodra@google.com> | 2018-02-16 13:20:56 -0800 |
---|---|---|
committer | Eric Erfanian <erfanian@google.com> | 2018-02-22 21:07:31 +0000 |
commit | 8074af74464a162e01296cc2b0b707f1c9d12bb7 (patch) | |
tree | 88c6901b4115ca79472367fb4085a958fa289364 /java/com/android/dialer/app | |
parent | ddd30052026e85de5237d4ef4751a4c26c09cb9b (diff) | |
download | android_packages_apps_Dialer-8074af74464a162e01296cc2b0b707f1c9d12bb7.tar.gz android_packages_apps_Dialer-8074af74464a162e01296cc2b0b707f1c9d12bb7.tar.bz2 android_packages_apps_Dialer-8074af74464a162e01296cc2b0b707f1c9d12bb7.zip |
Fix a few UI issues based around multiselect in new nav.
- No longer crashes when entering mw-mode while in multiselect
- Configuration changes now properly restore bottom nav visibility
- toolbar shadow is now only visible in search
Bug: 72525324
Test: MainActivityOldPeerSearchIntegrationTest
PiperOrigin-RevId: 186037379
Change-Id: I5fc00c8091e85fc67482b2131944fb776626d06f
Diffstat (limited to 'java/com/android/dialer/app')
-rw-r--r-- | java/com/android/dialer/app/calllog/CallLogFragment.java | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/java/com/android/dialer/app/calllog/CallLogFragment.java b/java/com/android/dialer/app/calllog/CallLogFragment.java index 7f635dbca..6b6b92297 100644 --- a/java/com/android/dialer/app/calllog/CallLogFragment.java +++ b/java/com/android/dialer/app/calllog/CallLogFragment.java @@ -661,8 +661,19 @@ public class CallLogFragment extends Fragment multiSelectUnSelectAllViewContent.setVisibility(show ? View.VISIBLE : View.GONE); multiSelectUnSelectAllViewContent.setAlpha(show ? 0 : 1); multiSelectUnSelectAllViewContent.animate().alpha(show ? 1 : 0).start(); - FragmentUtils.getParentUnsafe(this, CallLogFragmentListener.class) - .showMultiSelectRemoveView(show); + if (show) { + FragmentUtils.getParentUnsafe(this, CallLogFragmentListener.class) + .showMultiSelectRemoveView(true); + } else { + // This method is called after onDestroy. In DialtactsActivity, ListsFragment implements this + // interface and never goes away with configuration changes so this is safe. MainActivity + // removes that extra layer though, so we need to check if the parent is still there. + CallLogFragmentListener listener = + FragmentUtils.getParent(this, CallLogFragmentListener.class); + if (listener != null) { + listener.showMultiSelectRemoveView(false); + } + } } @Override |