commit 3884aab813b7932d43506a06dbca5343ef0234e4
author Mindy Pereira <mindyp@google.com> Wed Dec 11 10:18:13 2013 -0800
committer Andrew Sapperstein <asapperstein@google.com> Wed Feb 12 13:43:00 2014 -0800
tree 2186dafc26d3dfed78a1d8b783291655c462b694
parent 1e452a94e3ed24bad8c91e5f98387a9f3aa56e84

Enable swipe-refresh on empty labels. b/12095723.

Since SwipeRefreshLayout can have only one direct child,
wrap the list view and the empty view in a FrameLayout.

Change-Id: I0dbc636365da35ba51bd10990e023d73e4c9af1e

res/layout/conversation_list.xml

diff --git a/res/layout/conversation_list.xml b/res/layout/conversation_list.xml
index 561566a..85f5fb1 100644
--- a/res/layout/conversation_list.xml
+++ b/res/layout/conversation_list.xml
@@ -24,28 +24,33 @@
 
     <include layout="@layout/search_results_view"/>
 
-    <!-- Note: intentionally not called "empty" because we call
-             setEmptyView programmatically-->
-    <!-- Offset by height of undo bar and action bar -->
-    <com.android.mail.ui.EmptyConversationListView
-        android:id="@+id/empty_view"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent"
-        android:gravity="center"
-        android:textAppearance="?android:attr/textAppearanceLarge"
-        android:text="@string/no_conversations"
-        android:layout_marginTop="@dimen/empty_view_top_padding"
-        android:visibility="gone" />
-
     <android.support.v4.widget.SwipeRefreshLayout
         android:id="@+id/swipe_refresh_widget"
         android:layout_width="match_parent"
         android:layout_height="match_parent">
-        <com.android.mail.ui.SwipeableListView android:id="@android:id/list"
-            style="@style/ConversationListFade"
+        <FrameLayout
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            android:fadingEdge="none"
-            android:layout_alignParentTop="true"/>
+            android:clickable="true">
+            <!-- Note: intentionally not called "empty" because we call
+             setEmptyView programmatically-->
+            <!-- Offset by height of undo bar and action bar -->
+            <com.android.mail.ui.EmptyConversationListView
+                android:id="@+id/empty_view"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:layout_marginTop="@dimen/empty_view_top_padding"
+                android:gravity="center"
+                android:textAppearance="?android:attr/textAppearanceLarge"
+                android:text="@string/no_conversations"
+                android:visibility="gone"/>
+            <com.android.mail.ui.SwipeableListView
+                android:id="@android:id/list"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:layout_alignParentTop="true"
+                android:fadingEdge="none"
+                style="@style/ConversationListFade" />
+        </FrameLayout>
     </android.support.v4.widget.SwipeRefreshLayout>
 </RelativeLayout>