"Remove location" button for new uploads

app/src/main/java/fr/free/nrw/commons/LocationPicker/LocationPickerActivity.java

@@ -58,6 +58,7 @@
 import fr.free.nrw.commons.Utils;
 import fr.free.nrw.commons.kvstore.JsonKvStore;
 import fr.free.nrw.commons.theme.BaseActivity;
+import fr.free.nrw.commons.utils.DialogUtil;
 import fr.free.nrw.commons.utils.SystemThemeUtils;
 import javax.inject.Inject;
 import javax.inject.Named;
@@ -106,6 +107,10 @@ public class LocationPickerActivity extends BaseActivity implements OnMapReadyCa
      * modifyLocationButton : button for start editing location
      */
     Button modifyLocationButton;
+    /**
+     * removeLocationButton : button for removing location metadata
+     */
+    Button removeLocationButton;
     /**
      * showInMapButton : button for showing in map
      */
@@ -182,6 +187,7 @@ protected void onCreate(@Nullable final Bundle savedInstanceState) {
         if ("UploadActivity".equals(activity)) {
             placeSelectedButton.setVisibility(View.GONE);
             modifyLocationButton.setVisibility(View.VISIBLE);
+            removeLocationButton.setVisibility(View.VISIBLE);
             showInMapButton.setVisibility(View.VISIBLE);
             largeToolbarText.setText(getResources().getString(R.string.image_location));
             smallToolbarText.setText(getResources().
@@ -217,6 +223,7 @@ private void bindViews() {
         markerImage = findViewById(R.id.location_picker_image_view_marker);
         tvAttribution = findViewById(R.id.tv_attribution);
         modifyLocationButton = findViewById(R.id.modify_location);
+        removeLocationButton = findViewById(R.id.remove_location);
         showInMapButton = findViewById(R.id.show_in_map);
         showInMapButton.setText(getResources().getString(R.string.show_in_map_app).toUpperCase());
         shadow = findViewById(R.id.location_picker_image_view_shadow);
@@ -284,6 +291,7 @@ private void onStyleLoaded(final Style style) {
         }
 
         modifyLocationButton.setOnClickListener(v -> onClickModifyLocation());
+        removeLocationButton.setOnClickListener(v -> onClickRemoveLocation());
         showInMapButton.setOnClickListener(v -> showInMap());
     }
 
@@ -293,6 +301,7 @@ private void onStyleLoaded(final Style style) {
     private void onClickModifyLocation() {
         placeSelectedButton.setVisibility(View.VISIBLE);
         modifyLocationButton.setVisibility(View.GONE);
+        removeLocationButton.setVisibility(View.GONE);
         showInMapButton.setVisibility(View.GONE);
         droppedMarkerLayer.setProperties(visibility(NONE));
         markerImage.setVisibility(View.VISIBLE);
@@ -303,6 +312,47 @@ private void onClickModifyLocation() {
         fabCenterOnLocation.setVisibility(View.VISIBLE);
     }
 
+    /**
+     * Handles onclick event of removeLocationButton
+     */
+    private void onClickRemoveLocation() {
+        showRemovedLocationPopup();
+    }
+
+    /**
+     * Show the popup to confirm location removal
+     */
+    public void showRemovedLocationPopup() {
+        DialogUtil.showAlertDialog(this,
+            getString(R.string.no_location_selected),
+            getString(R.string.no_location_selected_warning_desc),
+            getString(R.string.continue_message),
+            getString(R.string.cancel), () -> removeLocationFromPicture(), null);
+    }
+
+    /**
+     * Method to remove the location from the picture and exit the location picker activity
+     */
+    private void removeLocationFromPicture() {
+        // Set the camera position to (0, 0)
+        cameraPosition = new CameraPosition.Builder()
+            .target(new LatLng(0, 0))
+            .zoom(16)
+            .build();
+
+        // Save the new location (0, 0) to preferences or any other data storage
+        applicationKvStore.putString(LAST_LOCATION,
+            mapboxMap.getCameraPosition().target.getLatitude() + ","
+                + mapboxMap.getCameraPosition().target.getLongitude());
+        applicationKvStore.putString(LAST_ZOOM, mapboxMap.getCameraPosition().zoom + "");
+
+        // Exit the location picker activity with the updated cameraPosition
+        final Intent returningIntent = new Intent();
+        returningIntent.putExtra(LocationPickerConstants.MAP_CAMERA_POSITION, cameraPosition);
+        setResult(AppCompatActivity.RESULT_OK, returningIntent);
+        finish();
+    }
+
     /**
      * Show the location in map app
      */

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

@@ -55,24 +55,46 @@
     app:layout_constraintStart_toStartOf="parent">
 
     <Button
-      android:id="@+id/modify_location"
+      android:id="@+id/remove_location"
+      style="@style/Widget.AppCompat.Button"
       android:layout_width="0dp"
       android:layout_height="wrap_content"
+      android:layout_margin="5dp"
+      android:backgroundTint="@color/deleteRed"
+      android:text="@string/remove_location"
+      android:textColor="@color/white"
+      android:visibility="gone"
+      app:layout_constraintBottom_toBottomOf="@id/map_bottom_layout"
+      app:layout_constraintEnd_toStartOf="@+id/guideline2"
+      app:layout_constraintStart_toStartOf="@id/map_bottom_layout"
+      app:layout_constraintTop_toTopOf="@id/map_bottom_layout" />
+
+    <androidx.constraintlayout.widget.Guideline
+      android:id="@+id/guideline2"
+      android:layout_width="wrap_content"
+      android:layout_height="wrap_content"
+      android:orientation="vertical"
+      app:layout_constraintGuide_percent="0.30" />
+
+    <Button
+      android:id="@+id/modify_location"
+      android:layout_width="0dp"
+      android:layout_height="0dp"
       android:text="@string/modify_location"
       android:textColor="@color/white"
       android:visibility="gone"
       android:layout_margin="5dp"
       app:layout_constraintBottom_toBottomOf="@id/map_bottom_layout"
       app:layout_constraintEnd_toStartOf="@+id/guideline3"
-      app:layout_constraintStart_toStartOf="@id/map_bottom_layout"
+      app:layout_constraintStart_toEndOf="@+id/guideline2"
       app:layout_constraintTop_toTopOf="@id/map_bottom_layout" />
 
     <androidx.constraintlayout.widget.Guideline
       android:id="@+id/guideline3"
       android:layout_width="wrap_content"
       android:layout_height="wrap_content"
       android:orientation="vertical"
-      app:layout_constraintGuide_percent="0.5" />
+      app:layout_constraintGuide_percent="0.70" />
 
     <TextView
       android:id="@+id/show_in_map"

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

@@ -423,6 +423,9 @@ Upload your first media by tapping on the add button.</string>
   <string name="no_depictions_selected">No Depictions Selected</string>
   <string name="no_depictions_selected_warning_desc">Images with depictions are more easily found and more likely to be used. Are you sure you want to continue without selecting depictions?</string>
 
+  <string name="no_location_selected">Removing Location</string>
+  <string name="no_location_selected_warning_desc">Location makes pictures more useful and findable. Do you really wish to remove location from this picture?</string>
+
   <string name="back_button_warning">Cancel Upload</string>
   <string name="back_button_warning_desc">Using the back button will cancel this upload and you will lose your progress</string>
   <string name="back_button_continue">Continue Upload</string>
@@ -683,6 +686,7 @@ Upload your first media by tapping on the add button.</string>
   <string name="select_location_location_picker">Select location</string>
   <string name="show_in_map_app">Show in map app</string>
   <string name="modify_location">Edit location</string>
+  <string name="remove_location">Remove location</string>
   <string name="location_picker_image_view">The image view of the location picker</string>
   <string name="location_picker_image_view_shadow">
     The shadow of the image view of the location picker</string>