From fdfeaaf4e5ae0277a03529e501eb1c562f769851 Mon Sep 17 00:00:00 2001 From: Andrew Lee Date: Tue, 5 May 2015 14:10:35 -0700 Subject: Change FAB icon and behavior for contacts tab. + Change the icon/behavior depending on the tab position. + Move current tab position logic from DialtactsActivity into the lists fragment. - Delete unused method, shift some helpers around. Bug: 19366434 Change-Id: I6da767300907b3afd006248afb882bebde7bdfe6 --- src/com/android/dialer/DialtactsActivity.java | 48 +++++++++++++++------------ 1 file changed, 27 insertions(+), 21 deletions(-) (limited to 'src/com/android/dialer/DialtactsActivity.java') diff --git a/src/com/android/dialer/DialtactsActivity.java b/src/com/android/dialer/DialtactsActivity.java index 167c131cb..70cf4a69c 100644 --- a/src/com/android/dialer/DialtactsActivity.java +++ b/src/com/android/dialer/DialtactsActivity.java @@ -197,11 +197,6 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O */ private boolean mIsLandscape; - /** - * The position of the currently selected tab in the attached {@link ListsFragment}. - */ - private int mCurrentTabPosition = 0; - /** * True if the dialpad is only temporarily showing due to being in call */ @@ -581,7 +576,9 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O public void onClick(View view) { switch (view.getId()) { case R.id.floating_action_button: - if (!mIsDialpadShown) { + if (mListsFragment.getTabPosition() == ListsFragment.TAB_INDEX_ALL_CONTACTS) { + sendAddNewContactIntent(); + } else if (!mIsDialpadShown) { mInCallDialpadUp = false; showDialpadFragment(true); } @@ -612,14 +609,7 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O showCallHistory(); break; case R.id.menu_add_contact: - try { - startActivity(new Intent(Intent.ACTION_INSERT, Contacts.CONTENT_URI)); - } catch (ActivityNotFoundException e) { - Toast toast = Toast.makeText(this, - R.string.add_contact_not_available, - Toast.LENGTH_SHORT); - toast.show(); - } + sendAddNewContactIntent(); break; case R.id.menu_import_export: // We hard-code the "contactsAreAvailable" argument because doing it properly would @@ -1094,6 +1084,17 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O return intent; } + private void sendAddNewContactIntent() { + try { + startActivity(new Intent(Intent.ACTION_INSERT, Contacts.CONTENT_URI)); + } catch (ActivityNotFoundException e) { + Toast toast = Toast.makeText(this, + R.string.add_contact_not_available, + Toast.LENGTH_SHORT); + toast.show(); + } + } + private boolean canIntentBeHandled(Intent intent) { final PackageManager packageManager = getPackageManager(); final List resolveInfo = packageManager.queryIntentActivities(intent, @@ -1194,22 +1195,26 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O } else if (position != ListsFragment.TAB_INDEX_SPEED_DIAL) { mFloatingActionButtonController.onPageScrolled(1); } + + if (position == ListsFragment.TAB_INDEX_ALL_CONTACTS) { + mFloatingActionButtonController.changeIcon( + getResources().getDrawable(R.drawable.ic_person_add_24dp), + getResources().getString(R.string.search_shortcut_create_new_contact)); + } else { + mFloatingActionButtonController.changeIcon( + getResources().getDrawable(R.drawable.fab_ic_dial), + getResources().getString(R.string.action_menu_dialpad_button)); + } } @Override public void onPageSelected(int position) { - position = mListsFragment.getRtlPosition(position); - mCurrentTabPosition = position; } @Override public void onPageScrollStateChanged(int state) { } - private TelephonyManager getTelephonyManager() { - return (TelephonyManager) getSystemService(Context.TELEPHONY_SERVICE); - } - private TelecomManager getTelecomManager() { return (TelecomManager) getSystemService(Context.TELECOM_SERVICE); } @@ -1249,7 +1254,8 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O * @param animate Whether or not to animate the transition. */ private void updateFloatingActionButtonControllerAlignment(boolean animate) { - int align = (!mIsLandscape && mCurrentTabPosition == ListsFragment.TAB_INDEX_SPEED_DIAL) ? + int align = (!mIsLandscape && + mListsFragment.getTabPosition() == ListsFragment.TAB_INDEX_SPEED_DIAL) ? FloatingActionButtonController.ALIGN_MIDDLE : FloatingActionButtonController.ALIGN_END; mFloatingActionButtonController.align(align, 0 /* offsetX */, 0 /* offsetY */, animate); -- cgit v1.2.3