Layout mExpansionViewContainer when not displayed
Force mExpansionViewContainer to always be laid out. This will ensure
that the whitespace on the end of mExpansionViewContainer is correct
regardless of whether an expansion view is needed or not.
This fixes the way phonetic name is displayed on exchange contacts.
This has no affect on focus contacts since both expansion views
are set to View.VISIBLE for focus contacts.
Bug: 19503535
Change-Id: I28aad309158a21a07d38a9699f82b48d72b8e7c4
diff --git a/src/com/android/contacts/editor/TextFieldsEditorView.java b/src/com/android/contacts/editor/TextFieldsEditorView.java
index d4fa45d..1798b0f 100644
--- a/src/com/android/contacts/editor/TextFieldsEditorView.java
+++ b/src/com/android/contacts/editor/TextFieldsEditorView.java
@@ -182,14 +182,10 @@
* Creates or removes the type/label button. Doesn't do anything if already correctly configured
*/
private void setupExpansionView(boolean shouldExist, boolean collapsed) {
- if (shouldExist) {
- mExpansionViewContainer.setVisibility(View.VISIBLE);
- mExpansionView.setImageResource(collapsed
- ? R.drawable.ic_menu_expander_minimized_holo_light
- : R.drawable.ic_menu_expander_maximized_holo_light);
- } else {
- mExpansionViewContainer.setVisibility(View.GONE);
- }
+ mExpansionView.setImageResource(collapsed
+ ? R.drawable.ic_menu_expander_minimized_holo_light
+ : R.drawable.ic_menu_expander_maximized_holo_light);
+ mExpansionViewContainer.setVisibility(shouldExist ? View.VISIBLE : View.INVISIBLE);
}
@Override