Remove show list FAB, since using multple fabs is not recommended. Use list button in menu instead

app/src/main/java/fr/free/nrw/commons/nearby/NearbyActivity.java

@@ -57,8 +57,6 @@ public class NearbyActivity extends NavigationBaseActivity implements LocationUp
     LinearLayout bottomSheet;
     @BindView(R.id.bottom_sheet_details)
     LinearLayout bottomSheetDetails;
-    @BindView(R.id.fab_list)
-    FloatingActionButton fabList;
     @BindView(R.id.transparentView)
     View transparentView;
 
@@ -87,8 +85,6 @@ public class NearbyActivity extends NavigationBaseActivity implements LocationUp
         bundle = new Bundle();
 
         initBottomSheetBehaviour();
-        initFabList();
-
         initDrawer();
         swipeLayout.setOnRefreshListener(new SwipeRefreshLayout.OnRefreshListener() {
             @Override
@@ -138,16 +134,6 @@ public class NearbyActivity extends NavigationBaseActivity implements LocationUp
         bottomSheetBehaviorForDetails.setState(BottomSheetBehavior.STATE_HIDDEN);
     }
 
-    private void initFabList() {
-        fabList.setOnClickListener(new View.OnClickListener() {
-            @Override
-            public void onClick(View view) {
-                bottomSheetBehaviorForDetails.setState(BottomSheetBehavior.STATE_HIDDEN);
-                bottomSheetBehavior.setState(BottomSheetBehavior.STATE_EXPANDED);
-            }
-        });
-    }
-
     @Override
     public boolean onCreateOptionsMenu(Menu menu) {
         MenuInflater inflater = getMenuInflater();
@@ -164,6 +150,10 @@ public class NearbyActivity extends NavigationBaseActivity implements LocationUp
                 lockNearbyView(false);
                 refreshView(true);
                 return true;
+            case R.id.action_display_list:
+                bottomSheetBehaviorForDetails.setState(BottomSheetBehavior.STATE_HIDDEN);
+                bottomSheetBehavior.setState(BottomSheetBehavior.STATE_EXPANDED);
+                return true;
             default:
                 return super.onOptionsItemSelected(item);
         }
@@ -422,11 +412,6 @@ public class NearbyActivity extends NavigationBaseActivity implements LocationUp
     }
 
     public void prepareViewsForSheetPosition(int bottomSheetState) {
-        if (bottomSheetState >= BottomSheetBehavior.STATE_COLLAPSED) {
+        // TODO
-            fabList.show();
-        }
-        else if (bottomSheetState == BottomSheetBehavior.STATE_EXPANDED) {
-            fabList.hide();
-        }
     }
 }

app/src/main/java/fr/free/nrw/commons/nearby/NearbyMapFragment.java

@@ -56,7 +56,6 @@ public class NearbyMapFragment extends android.support.v4.app.Fragment {
     private LinearLayout wikipediaButton;
     private LinearLayout wikidataButton;
     private LinearLayout directionsButton;
-    private FloatingActionButton fabList;
     private FloatingActionButton fabPlus;
     private FloatingActionButton fabCamera;
     private FloatingActionButton fabGallery;
@@ -148,7 +147,6 @@ public class NearbyMapFragment extends android.support.v4.app.Fragment {
         bottomSheetDetailsBehavior = BottomSheetBehavior.from(bottomSheetDetails);
         bottomSheetDetailsBehavior.setState(BottomSheetBehavior.STATE_HIDDEN);
 
-        fabList = getActivity().findViewById(R.id.fab_list);
         fabPlus = getActivity().findViewById(R.id.fab_plus);
         fabCamera = getActivity().findViewById(R.id.fab_camera);
         fabGallery = getActivity().findViewById(R.id.fab_galery);
@@ -311,21 +309,18 @@ public class NearbyMapFragment extends android.support.v4.app.Fragment {
 
         switch (bottomSheetState) {
             case (BottomSheetBehavior.STATE_COLLAPSED):
-                if (!fabList.isShown()) fabList.show();
                 closeFabs(isFabOpen);
                 if (!fabPlus.isShown()) showFAB();
                 this.getView().requestFocus();
                 moreInfo.setVisibility(View.VISIBLE);
                 break;
             case (BottomSheetBehavior.STATE_EXPANDED):
-                if (fabList.isShown()) fabList.hide();
                 this.getView().requestFocus();
                 moreInfo.setVisibility(View.VISIBLE);
                 break;
             case (BottomSheetBehavior.STATE_HIDDEN):
                 transparentView.setClickable(false);
                 transparentView.setAlpha(0);
-                if (!fabList.isShown()) fabList.show();
                 closeFabs(isFabOpen);
                 hideFAB();
                 this.getView().requestFocus();

app/src/main/res/layout/activity_nearby.xml

@@ -20,12 +20,12 @@
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content" />
 
-        <LinearLayout
+            <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-            android:layout_width="match_parent"
+                android:layout_width="match_parent"
-            android:layout_height="match_parent"
+                android:layout_height="match_parent"
-            android:orientation="horizontal"
+                android:layout_below="@id/toolbar"
-            android:gravity="center_vertical"
+                android:gravity="center_vertical"
-            android:layout_below="@id/toolbar">
+                android:orientation="horizontal">
             <ProgressBar
                 android:id="@+id/progressBar"
                 android:layout_width="match_parent"

app/src/main/res/menu/menu_nearby.xml

@@ -2,6 +2,14 @@
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"      tools:context=".MainActivity">
 
+    <item
+        android:id="@+id/action_display_list"
+        android:title="@string/display_list_button"
+        android:icon="@drawable/ic_list_white_24dp"
+        android:orderInCategory="1"
+        app:showAsAction="ifRoom"
+        />
+
     <item
         android:id="@+id/action_refresh"
         android:title="@string/refresh_button"

app/src/main/res/values/strings.xml

@@ -53,6 +53,7 @@
   <string name="categories_search_text_hint">Search categories</string>
   <string name="menu_save_categories">Save</string>
   <string name="refresh_button">Refresh</string>
+  <string name="display_list_button">List</string>
   <string name="gps_disabled">GPS is disabled in your device. Would you like to enable it?</string>
   <string name="enable_gps">Enable GPS</string>
   <string name="contributions_subtitle_zero">No uploads yet</string>