Remove the expand/collapse behavior of the QuickContact contact card
Also remove the prioritization for showing the
EXTRA_PRIORITIZED_MIMETYPE, which is used only for collapsed contact
card.
bug: 30979328
bug: 26063493
test: Manually opened QuickContact from Contacts and Dialer.
Manually added/removed contact information to have less than 3, 3,
and more than 3 entries shown in contact card.
Change-Id: Ifeabc7012306a5c35e8572a823cb1b7004218409
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 14207d2..04f2b0d 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -712,8 +712,6 @@
<string name="expanding_entry_card_view_see_more">See more</string>
<!-- Button Label to see less on an ExpandingEntryCardView [CHAR LIMIT=40] -->
<string name="expanding_entry_card_view_see_less">See less</string>
- <!-- Button Label to see all on an ExpandingEntryCardView [CHAR LIMIT=40] -->
- <string name="expanding_entry_card_view_see_all">See all</string>
<!-- Title of recent card. [CHAR LIMIT=60] -->
<string name="recent_card_title">Recent</string>
diff --git a/src/com/android/contacts/quickcontact/ExpandingEntryCardView.java b/src/com/android/contacts/quickcontact/ExpandingEntryCardView.java
index da017cc..018f468 100644
--- a/src/com/android/contacts/quickcontact/ExpandingEntryCardView.java
+++ b/src/com/android/contacts/quickcontact/ExpandingEntryCardView.java
@@ -244,8 +244,6 @@
private View mExpandCollapseButton;
private TextView mExpandCollapseTextView;
private TextView mTitleTextView;
- private CharSequence mExpandButtonText;
- private CharSequence mCollapseButtonText;
private OnClickListener mOnClickListener;
private OnCreateContextMenuListener mOnCreateContextMenuListener;
private boolean mIsExpanded = false;
@@ -263,11 +261,6 @@
private final ImageView mExpandCollapseArrow;
private int mThemeColor;
private ColorFilter mThemeColorFilter;
- /**
- * Whether to prioritize the first entry type. If prioritized, we should show at least two
- * of this entry type.
- */
- private boolean mShowFirstEntryTypeTwice;
private boolean mIsAlwaysExpanded;
/** The ViewGroup to run the expand/collapse animation on */
private ViewGroup mAnimationViewGroup;
@@ -319,22 +312,14 @@
mBadgeIds = new ArrayList<Integer>();
}
- public void initialize(List<List<Entry>> entries, int numInitialVisibleEntries,
- boolean isExpanded, boolean isAlwaysExpanded, ExpandingEntryCardViewListener listener,
- ViewGroup animationViewGroup) {
- initialize(entries, numInitialVisibleEntries, isExpanded, isAlwaysExpanded,
- listener, animationViewGroup, /* showFirstEntryTypeTwice = */ false);
- }
-
/**
* Sets the Entry list to display.
*
* @param entries The Entry list to display.
*/
public void initialize(List<List<Entry>> entries, int numInitialVisibleEntries,
- boolean isExpanded, boolean isAlwaysExpanded,
- ExpandingEntryCardViewListener listener, ViewGroup animationViewGroup,
- boolean showFirstEntryTypeTwice) {
+ boolean isExpanded, boolean isAlwaysExpanded, ExpandingEntryCardViewListener listener,
+ ViewGroup animationViewGroup) {
LayoutInflater layoutInflater = LayoutInflater.from(getContext());
mIsExpanded = isExpanded;
mIsAlwaysExpanded = isAlwaysExpanded;
@@ -344,7 +329,6 @@
mEntries = entries;
mNumEntries = 0;
mAllEntriesInflated = false;
- mShowFirstEntryTypeTwice = showFirstEntryTypeTwice;
for (List<Entry> entryList : mEntries) {
mNumEntries += entryList.size();
mEntryViews.add(new ArrayList<View>());
@@ -368,30 +352,6 @@
applyColor();
}
- /**
- * Sets the text for the expand button.
- *
- * @param expandButtonText The expand button text.
- */
- public void setExpandButtonText(CharSequence expandButtonText) {
- mExpandButtonText = expandButtonText;
- if (mExpandCollapseTextView != null && !mIsExpanded) {
- mExpandCollapseTextView.setText(expandButtonText);
- }
- }
-
- /**
- * Sets the text for the expand button.
- *
- * @param expandButtonText The expand button text.
- */
- public void setCollapseButtonText(CharSequence expandButtonText) {
- mCollapseButtonText = expandButtonText;
- if (mExpandCollapseTextView != null && mIsExpanded) {
- mExpandCollapseTextView.setText(mCollapseButtonText);
- }
- }
-
@Override
public void setOnClickListener(OnClickListener listener) {
mOnClickListener = listener;
@@ -468,19 +428,9 @@
viewsToDisplay.add(entryViewList.get(0));
numInViewGroup++;
- int indexInEntryViewList = 1;
- if (mShowFirstEntryTypeTwice && i == 0 && entryViewList.size() > 1) {
- viewsToDisplay.add(entryViewList.get(1));
- numInViewGroup++;
- extraEntries--;
- indexInEntryViewList++;
- }
-
// Insert entries in this list to hit mCollapsedEntriesCount.
- for (int j = indexInEntryViewList;
- j < entryViewList.size() && numInViewGroup < mCollapsedEntriesCount &&
- extraEntries > 0;
- j++) {
+ for (int j = 1; j < entryViewList.size() && numInViewGroup < mCollapsedEntriesCount
+ && extraEntries > 0; j++) {
viewsToDisplay.add(entryViewList.get(j));
numInViewGroup++;
extraEntries--;
@@ -532,21 +482,13 @@
}
private CharSequence getExpandButtonText() {
- if (!TextUtils.isEmpty(mExpandButtonText)) {
- return mExpandButtonText;
- } else {
- // Default to "See more".
- return getResources().getText(R.string.expanding_entry_card_view_see_more);
- }
+ // Default to "See more".
+ return getResources().getText(R.string.expanding_entry_card_view_see_more);
}
private CharSequence getCollapseButtonText() {
- if (!TextUtils.isEmpty(mCollapseButtonText)) {
- return mCollapseButtonText;
- } else {
- // Default to "See less".
- return getResources().getText(R.string.expanding_entry_card_view_see_less);
- }
+ // Default to "See less".
+ return getResources().getText(R.string.expanding_entry_card_view_see_less);
}
/**
@@ -569,18 +511,8 @@
/* showIcon = */ View.VISIBLE));
numInflated++;
- int indexInEntryViewList = 1;
- if (mShowFirstEntryTypeTwice && i == 0 && entryList.size() > 1) {
- entryViewList.add(createEntryView(layoutInflater, entryList.get(1),
- /* showIcon = */ View.INVISIBLE));
- numInflated++;
- extraEntries--;
- indexInEntryViewList++;
- }
-
// Inflate entries in this list to hit mCollapsedEntriesCount.
- for (int j = indexInEntryViewList; j < entryList.size()
- && numInflated < mCollapsedEntriesCount
+ for (int j = 1; j < entryList.size() && numInflated < mCollapsedEntriesCount
&& extraEntries > 0; j++) {
entryViewList.add(createEntryView(layoutInflater, entryList.get(j),
/* showIcon = */ View.INVISIBLE));
@@ -851,10 +783,7 @@
mBadgeContainer.removeAllViews();
} else {
int numberOfMimeTypesShown = mCollapsedEntriesCount;
- if (mShowFirstEntryTypeTwice && mEntries.size() > 0
- && mEntries.get(0).size() > 1) {
- numberOfMimeTypesShown--;
- }
+
// Inflate badges if not yet created
if (mBadges.size() < mEntries.size() - numberOfMimeTypesShown) {
for (int i = numberOfMimeTypesShown; i < mEntries.size(); i++) {
diff --git a/src/com/android/contacts/quickcontact/QuickContactActivity.java b/src/com/android/contacts/quickcontact/QuickContactActivity.java
index fc43b9e..73c3c78 100644
--- a/src/com/android/contacts/quickcontact/QuickContactActivity.java
+++ b/src/com/android/contacts/quickcontact/QuickContactActivity.java
@@ -1106,8 +1106,6 @@
mPermissionExplanationCard.setOnClickListener(mEntryClickHandler);
mNoContactDetailsCard.setOnClickListener(mEntryClickHandler);
mContactCard.setOnClickListener(mEntryClickHandler);
- mContactCard.setExpandButtonText(
- getResources().getString(R.string.expanding_entry_card_view_see_all));
mContactCard.setOnCreateContextMenuListener(mEntryContextMenuListener);
mRecentCard.setOnClickListener(mEntryClickHandler);
@@ -1593,17 +1591,12 @@
final String customAboutCardName = cp2DataCardModel.customAboutCardName;
if (contactCardEntries.size() > 0) {
- final boolean firstEntriesArePrioritizedMimeType =
- !TextUtils.isEmpty(mExtraPrioritizedMimeType) &&
- mCachedCp2DataCardModel.dataItemsMap.containsKey(mExtraPrioritizedMimeType) &&
- mCachedCp2DataCardModel.dataItemsMap.get(mExtraPrioritizedMimeType).size() != 0;
mContactCard.initialize(contactCardEntries,
/* numInitialVisibleEntries = */ MIN_NUM_CONTACT_ENTRIES_SHOWN,
/* isExpanded = */ mContactCard.isExpanded(),
- /* isAlwaysExpanded = */ false,
+ /* isAlwaysExpanded = */ true,
mExpandingEntryCardViewListener,
- mScroller,
- firstEntriesArePrioritizedMimeType);
+ mScroller);
if (mContactCard.getVisibility() == View.GONE && mShouldLog) {
Logger.logQuickContactEvent(mReferrer, mContactType, CardType.CONTACT,
ActionType.UNKNOWN_ACTION, /* thirdPartyAction */ null);