From e42c6feba45d013abf5714b2c1c54503dfcb453e Mon Sep 17 00:00:00 2001
From: Suchit Kar <suchitkar1997@gmail.com>
Date: Mon, 19 Mar 2018 23:20:46 +0530
Subject: [PATCH 1/5] Check status before nominating for deletion

---
 .../main/java/fr/free/nrw/commons/Media.java  |  9 +++
 .../free/nrw/commons/MediaDataExtractor.java  | 12 ++++
 .../free/nrw/commons/delete/DeleteTask.java   | 67 ++++++++-----------
 .../commons/media/MediaDetailFragment.java    | 34 ++++++++--
 .../main/res/drawable/bg_delete_button.xml    |  7 ++
 .../main/res/layout/fragment_media_detail.xml | 18 ++++-
 app/src/main/res/values/colors.xml            |  3 +-
 app/src/main/res/values/strings.xml           |  1 +
 8 files changed, 101 insertions(+), 50 deletions(-)
 create mode 100644 app/src/main/res/drawable/bg_delete_button.xml

diff --git a/app/src/main/java/fr/free/nrw/commons/Media.java b/app/src/main/java/fr/free/nrw/commons/Media.java
index 726d787f3..5f6a498ea 100644
--- a/app/src/main/java/fr/free/nrw/commons/Media.java
+++ b/app/src/main/java/fr/free/nrw/commons/Media.java
@@ -43,6 +43,7 @@ public class Media implements Parcelable {
     protected String license;
     protected String creator;
     protected ArrayList<String> categories; // as loaded at runtime?
+    protected boolean requestedDeletion;
     private Map<String, String> descriptions; // multilingual descriptions as loaded
     private HashMap<String, Object> tags = new HashMap<>();
     private @Nullable LatLng coordinates;
@@ -416,4 +417,12 @@ public class Media implements Parcelable {
         parcel.writeStringList(categories);
         parcel.writeMap(descriptions);
     }
+
+    public void setRequestedDeletion(){
+        requestedDeletion = true;
+    }
+
+    public boolean getRequestedDeletion(){
+        return requestedDeletion;
+    }
 }
diff --git a/app/src/main/java/fr/free/nrw/commons/MediaDataExtractor.java b/app/src/main/java/fr/free/nrw/commons/MediaDataExtractor.java
index 25e778b74..2d79a6c4f 100644
--- a/app/src/main/java/fr/free/nrw/commons/MediaDataExtractor.java
+++ b/app/src/main/java/fr/free/nrw/commons/MediaDataExtractor.java
@@ -35,6 +35,7 @@ import timber.log.Timber;
 public class MediaDataExtractor {
     private final MediaWikiApi mediaWikiApi;
     private boolean fetched;
+    private boolean deletionStatus;
     private ArrayList<String> categories;
     private Map<String, String> descriptions;
     private String license;
@@ -59,6 +60,14 @@ public class MediaDataExtractor {
             throw new IllegalStateException("Tried to call MediaDataExtractor.fetch() again.");
         }
 
+        try{
+            Timber.d("Nominated for deletion: " + mediaWikiApi.pageExists("Commons:Deletion_requests/"+filename));
+            deletionStatus = mediaWikiApi.pageExists("Commons:Deletion_requests/"+filename);
+        }
+        catch (Exception e){
+            Timber.d(e.getMessage());
+        }
+
         MediaResult result = mediaWikiApi.fetchMediaByFilename(filename);
 
         // In-page category links are extracted from source, as XML doesn't cover [[links]]
@@ -296,6 +305,9 @@ public class MediaDataExtractor {
         if (license != null) {
             media.setLicense(license);
         }
+        if (deletionStatus){
+            media.setRequestedDeletion();
+        }
 
         // add author, date, etc fields
     }
diff --git a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
index 07ae8cb81..c75c05e69 100644
--- a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
+++ b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
@@ -5,6 +5,8 @@ import android.content.Context;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.os.AsyncTask;
+import android.widget.Toast;
+
 import java.text.SimpleDateFormat;
 import java.util.Calendar;
 import java.util.Locale;
@@ -19,12 +21,9 @@ import fr.free.nrw.commons.mwapi.MediaWikiApi;
 import timber.log.Timber;
 
 import static android.support.v4.content.ContextCompat.startActivity;
+import static android.widget.Toast.LENGTH_SHORT;
 
-public class DeleteTask extends AsyncTask<Void, Void, Integer> {
-
-    private static final int SUCCESS = 0;
-    private static final int FAILED = -1;
-    private static final int ALREADY_DELETED = -2;
+public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
 
     @Inject MediaWikiApi mwApi;
     @Inject SessionManager sessionManager;
@@ -48,7 +47,7 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
     }
 
     @Override
-    protected Integer doInBackground(Void ...voids) {
+    protected Boolean doInBackground(Void ...voids) {
         String editToken;
         String authCookie;
         String summary = "Nominating " + media.getFilename() +" for deletion.";
@@ -56,25 +55,15 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
         authCookie = sessionManager.getAuthCookie();
         mwApi.setAuthCookie(authCookie);
 
-        try{
-            if (mwApi.pageExists("Commons:Deletion_requests/"+media.getFilename())){
-                return ALREADY_DELETED;
-            }
-        }
-        catch (Exception e) {
-            Timber.d(e.getMessage());
-            return FAILED;
-        }
-
         try {
             editToken = mwApi.getEditToken();
         }
         catch (Exception e){
             Timber.d(e.getMessage());
-            return FAILED;
+            return false;
         }
         if (editToken.equals("+\\")) {
-            return FAILED;
+            return false;
         }
 
         Calendar calendar = Calendar.getInstance();
@@ -90,7 +79,7 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
         }
         catch (Exception e) {
             Timber.d(e.getMessage());
-            return FAILED;
+            return false;
         }
 
         String subpageString = "=== [[:" + media.getFilename() + "]] ===\n" +
@@ -102,7 +91,7 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
         }
         catch (Exception e) {
             Timber.d(e.getMessage());
-            return FAILED;
+            return false;
         }
 
         String logPageString = "\n{{Commons:Deletion requests/" + media.getFilename() +
@@ -115,7 +104,7 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
         }
         catch (Exception e) {
             Timber.d(e.getMessage());
-            return FAILED;
+            return false;
         }
 
         String userPageString = "\n{{subst:idw|" + media.getFilename() +
@@ -126,30 +115,23 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
         }
         catch (Exception e) {
             Timber.d(e.getMessage());
-            return FAILED;
+            return false;
         }
-        return SUCCESS;
+        return true;
     }
 
     @Override
-    protected void onPostExecute(Integer result) {
+    protected void onPostExecute(Boolean result) {
         String message = "";
         String title = "";
-        switch (result){
-            case SUCCESS:
-                title = "Success";
-                message = "Successfully nominated " + media.getDisplayTitle() + " deletion.\n" +
-                        "Check the webpage for more details";
-                break;
-            case FAILED:
-                title = "Failed";
-                message = "Could not request deletion. Something went wrong.";
-                break;
-            case ALREADY_DELETED:
-                title = "Already Nominated";
-                message = media.getDisplayTitle() + " has already been nominated for deletion.\n" +
-                        "Check the webpage for more details";
-                break;
+        if (result){
+            title = "Success";
+            message = "Successfully nominated " + media.getDisplayTitle() + " deletion.\n" +
+                    "Check the webpage for more details";
+        }
+        else {
+            title = "Failed";
+            message = "Could not request deletion. Something went wrong.";
         }
         AlertDialog alert;
         AlertDialog.Builder builder = new AlertDialog.Builder(context);
@@ -165,7 +147,12 @@ public class DeleteTask extends AsyncTask<Void, Void, Integer> {
                 new DialogInterface.OnClickListener() {
                     public void onClick(DialogInterface dialog, int id) {
                         Intent browserIntent = new Intent(Intent.ACTION_VIEW, media.getFilePageTitle().getMobileUri());
-                        startActivity(context,browserIntent,null);
+                        if (browserIntent.resolveActivity(context.getPackageManager()) != null) {
+                            startActivity(context,browserIntent,null);
+                        } else {
+                            Toast toast = Toast.makeText(context, R.string.no_web_browser, LENGTH_SHORT);
+                            toast.show();
+                        }
                     }
                 });
         alert = builder.create();
diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
index 2407602bb..c070fb680 100644
--- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
@@ -15,7 +15,6 @@ import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.ViewTreeObserver;
-import android.view.WindowManager;
 import android.widget.Button;
 import android.widget.EditText;
 import android.widget.LinearLayout;
@@ -42,6 +41,7 @@ import fr.free.nrw.commons.R;
 import fr.free.nrw.commons.delete.DeleteTask;
 import fr.free.nrw.commons.di.CommonsDaggerSupportFragment;
 import fr.free.nrw.commons.location.LatLng;
+import fr.free.nrw.commons.mwapi.MediaWikiApi;
 import fr.free.nrw.commons.ui.widget.CompatTextView;
 import timber.log.Timber;
 
@@ -69,6 +69,9 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
 
     @Inject
     Provider<MediaDataExtractor> mediaDataExtractorProvider;
+    @Inject
+    MediaWikiApi mwApi;
+
 
     private MediaWikiImageView image;
     private MediaDetailSpacer spacer;
@@ -79,6 +82,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
     private TextView license;
     private TextView coordinates;
     private TextView uploadedDate;
+    private TextView nominatedforDeletion;
     private LinearLayout categoryContainer;
     private Button delete;
     private ScrollView scrollView;
@@ -133,6 +137,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
         license = (TextView) view.findViewById(R.id.mediaDetailLicense);
         coordinates = (TextView) view.findViewById(R.id.mediaDetailCoordinates);
         uploadedDate = (TextView) view.findViewById(R.id.mediaDetailuploadeddate);
+        nominatedforDeletion = (TextView) view.findViewById(R.id.nominatedDeletionBanner);
         delete = (Button) view.findViewById(R.id.nominateDeletion);
         categoryContainer = (LinearLayout) view.findViewById(R.id.mediaDetailCategoryContainer);
 
@@ -231,7 +236,6 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
 
                 if (success) {
                     extractor.fill(media);
-
                     setTextFields(media);
                     setOnClickListeners(media);
                 } else {
@@ -284,21 +288,22 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
         }
         rebuildCatList();
 
-        delete.setVisibility(View.VISIBLE);
+        checkDeletion(media);
     }
 
     private void setOnClickListeners(final Media media) {
         if (licenseLink(media) != null) {
             license.setOnClickListener(v -> openWebBrowser(licenseLink(media)));
-            } else {
+        } else {
             Toast toast = Toast.makeText(getContext(), getString(R.string.null_url), Toast.LENGTH_SHORT);
             toast.show();
-            }
+        }
         if (media.getCoordinates() != null) {
             coordinates.setOnClickListener(v -> openMap(media.getCoordinates()));
         }
-        if (delete.getVisibility()==View.VISIBLE){
+        if (delete.getVisibility() == View.VISIBLE) {
             delete.setOnClickListener(v -> {
+                delete.setEnabled(false);
                 AlertDialog.Builder alert = new AlertDialog.Builder(getActivity());
                 alert.setMessage("Why should this file be deleted?");
                 final EditText input = new EditText(getActivity());
@@ -343,6 +348,11 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
                 d.getButton(AlertDialog.BUTTON_POSITIVE).setEnabled(false);
             });
         }
+        if (nominatedforDeletion.getVisibility() == View.VISIBLE){
+            nominatedforDeletion.setOnClickListener(v -> {
+                openWebBrowser(media.getFilePageTitle().getMobileUri().toString());
+            });
+        }
     }
 
     private void rebuildCatList() {
@@ -366,7 +376,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
                 viewIntent.setAction(Intent.ACTION_VIEW);
                 viewIntent.setData(new PageTitle(selectedCategoryTitle).getCanonicalUri());
                 //check if web browser available
-                if(viewIntent.resolveActivity(getActivity().getPackageManager()) != null){
+                if (viewIntent.resolveActivity(getActivity().getPackageManager()) != null) {
                     startActivity(viewIntent);
                 } else {
                     Toast toast = Toast.makeText(getContext(), getString(R.string.no_web_browser), LENGTH_SHORT);
@@ -434,6 +444,16 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
         return media.getCoordinates().getPrettyCoordinateString();
     }
 
+    private void checkDeletion(Media media){
+        if (media.getRequestedDeletion()){
+            delete.setVisibility(View.GONE);
+            nominatedforDeletion.setVisibility(View.VISIBLE);
+        }
+        else{
+            delete.setVisibility(View.VISIBLE);
+            nominatedforDeletion.setVisibility(View.GONE);
+        }
+    }
 
     private @Nullable
     String licenseLink(Media media) {
diff --git a/app/src/main/res/drawable/bg_delete_button.xml b/app/src/main/res/drawable/bg_delete_button.xml
new file mode 100644
index 000000000..8fef090ab
--- /dev/null
+++ b/app/src/main/res/drawable/bg_delete_button.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle">
+    <solid android:color="@color/deleteButtonColor"/>
+    <corners android:radius="3dp" />
+    <stroke android:width="5px" android:color="@color/deleteBackgroundColor" />
+</shape>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_media_detail.xml b/app/src/main/res/layout/fragment_media_detail.xml
index 4881c2061..e7db8b694 100644
--- a/app/src/main/res/layout/fragment_media_detail.xml
+++ b/app/src/main/res/layout/fragment_media_detail.xml
@@ -251,10 +251,24 @@
                     android:layout_width="match_parent"
                     android:layout_height="@dimen/small_gap" />
 
+                <TextView
+                    android:id="@+id/nominatedDeletionBanner"
+                    android:background="@color/deleteBackgroundColor"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:paddingBottom="@dimen/tiny_gap"
+                    android:gravity="center"
+                    android:text="@string/nominated_for_deletion"
+                    android:textColor="@color/primaryTextColor"
+                    android:padding="@dimen/standard_gap"
+                    android:textSize="@dimen/normal_text"
+                    android:textStyle="bold"
+                    android:visibility="gone"/>
+
                 <Button
                     android:id="@+id/nominateDeletion"
-                    style="?android:attr/buttonBarButtonStyle"
-                    android:textColor="@color/deleteTextColor"
+                    android:textColor="@color/primaryTextColor"
+                    android:background="@drawable/bg_delete_button"
                     android:layout_margin="@dimen/standard_gap"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index 8e79f581e..4ce597181 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -15,7 +15,8 @@
     <color name="primaryTextColor">#ffffff</color>
     <color name="secondaryTextColor">#000000</color>
 
-    <color name="deleteTextColor">#d50000</color>
+    <color name="deleteBackgroundColor">#90960a0a</color>
+    <color name="deleteButtonColor">#44000000</color>
 
     <!-- Some colours are same for dark/light themes. They are written two times in case
     we want to change light ones later.
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 17fbd6ff9..980bf6ac7 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -231,6 +231,7 @@
   <string name="no_web_browser">No web browser found to open URL</string>
   <string name="null_url">Error! URL not found</string>
   <string name="nominate_deletion">Nominate for Deletion</string>
+  <string name="nominated_for_deletion">This image has been nominated for deletion.\n\n<u>See webpage for details</u></string>
   <string name="view_browser">View in Browser</string>
 
   <string name="nearby_location_has_not_changed">Location has not changed.</string>

From a39d398edd292588407a2298b09d2939da4c3161 Mon Sep 17 00:00:00 2001
From: Suchit Kar <suchitkar1997@gmail.com>
Date: Tue, 20 Mar 2018 15:32:50 +0530
Subject: [PATCH 2/5] Make deletion-related views on MediaDetailFragment neater

---
 .../commons/media/MediaDetailFragment.java    |  8 +++--
 .../main/res/layout/fragment_media_detail.xml | 30 ++++++++++++++-----
 app/src/main/res/values/strings.xml           |  3 +-
 3 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
index c070fb680..b54bf948e 100644
--- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
@@ -82,7 +82,8 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
     private TextView license;
     private TextView coordinates;
     private TextView uploadedDate;
-    private TextView nominatedforDeletion;
+    private TextView seeMore;
+    private LinearLayout nominatedforDeletion;
     private LinearLayout categoryContainer;
     private Button delete;
     private ScrollView scrollView;
@@ -137,7 +138,8 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
         license = (TextView) view.findViewById(R.id.mediaDetailLicense);
         coordinates = (TextView) view.findViewById(R.id.mediaDetailCoordinates);
         uploadedDate = (TextView) view.findViewById(R.id.mediaDetailuploadeddate);
-        nominatedforDeletion = (TextView) view.findViewById(R.id.nominatedDeletionBanner);
+        seeMore = (TextView) view.findViewById(R.id.seeMore);
+        nominatedforDeletion = (LinearLayout) view.findViewById(R.id.nominatedDeletionBanner);
         delete = (Button) view.findViewById(R.id.nominateDeletion);
         categoryContainer = (LinearLayout) view.findViewById(R.id.mediaDetailCategoryContainer);
 
@@ -349,7 +351,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
             });
         }
         if (nominatedforDeletion.getVisibility() == View.VISIBLE){
-            nominatedforDeletion.setOnClickListener(v -> {
+            seeMore.setOnClickListener(v -> {
                 openWebBrowser(media.getFilePageTitle().getMobileUri().toString());
             });
         }
diff --git a/app/src/main/res/layout/fragment_media_detail.xml b/app/src/main/res/layout/fragment_media_detail.xml
index e7db8b694..2b6ecf314 100644
--- a/app/src/main/res/layout/fragment_media_detail.xml
+++ b/app/src/main/res/layout/fragment_media_detail.xml
@@ -251,19 +251,33 @@
                     android:layout_width="match_parent"
                     android:layout_height="@dimen/small_gap" />
 
-                <TextView
+                <LinearLayout
                     android:id="@+id/nominatedDeletionBanner"
                     android:background="@color/deleteBackgroundColor"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
-                    android:paddingBottom="@dimen/tiny_gap"
-                    android:gravity="center"
-                    android:text="@string/nominated_for_deletion"
-                    android:textColor="@color/primaryTextColor"
+                    android:orientation="vertical"
                     android:padding="@dimen/standard_gap"
-                    android:textSize="@dimen/normal_text"
-                    android:textStyle="bold"
-                    android:visibility="gone"/>
+                    android:visibility="gone">
+                    <TextView
+                        android:layout_width="match_parent"
+                        android:layout_height="wrap_content"
+                        android:gravity="center"
+                        android:text="@string/nominated_for_deletion"
+                        android:textColor="@color/primaryTextColor"
+                        android:textSize="@dimen/normal_text"
+                        android:textStyle="bold"/>
+                    <TextView
+                        android:id="@+id/seeMore"
+                        android:layout_width="match_parent"
+                        android:layout_height="wrap_content"
+                        android:gravity="center"
+                        android:text="@string/nominated_see_more"
+                        android:paddingTop="@dimen/standard_gap"
+                        android:textColor="@color/primaryTextColor"
+                        android:textSize="@dimen/normal_text"
+                        android:textStyle="bold"/>
+                </LinearLayout>
 
                 <Button
                     android:id="@+id/nominateDeletion"
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 980bf6ac7..600fc3728 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -231,7 +231,8 @@
   <string name="no_web_browser">No web browser found to open URL</string>
   <string name="null_url">Error! URL not found</string>
   <string name="nominate_deletion">Nominate for Deletion</string>
-  <string name="nominated_for_deletion">This image has been nominated for deletion.\n\n<u>See webpage for details</u></string>
+  <string name="nominated_for_deletion">This image has been nominated for deletion.</string>
+  <string name="nominated_see_more"><u>See webpage for details</u></string>
   <string name="view_browser">View in Browser</string>
 
   <string name="nearby_location_has_not_changed">Location has not changed.</string>

From a449870d8b45dd16e945edab48c3638eab08bad9 Mon Sep 17 00:00:00 2001
From: Suchit Kar <suchitkar1997@gmail.com>
Date: Tue, 20 Mar 2018 19:46:24 +0530
Subject: [PATCH 3/5] Change disabled appearance of Nominate for Deletion
 button

---
 .../commons/media/MediaDetailFragment.java    |  1 +
 .../main/res/drawable/bg_delete_button.xml    | 37 ++++++++++++++++---
 .../main/res/layout/fragment_media_detail.xml |  4 +-
 app/src/main/res/values/colors.xml            |  6 ++-
 4 files changed, 38 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
index b54bf948e..b06869e8d 100644
--- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailFragment.java
@@ -306,6 +306,7 @@ public class MediaDetailFragment extends CommonsDaggerSupportFragment {
         if (delete.getVisibility() == View.VISIBLE) {
             delete.setOnClickListener(v -> {
                 delete.setEnabled(false);
+                delete.setTextColor(getResources().getColor(R.color.deleteButtonLight));
                 AlertDialog.Builder alert = new AlertDialog.Builder(getActivity());
                 alert.setMessage("Why should this file be deleted?");
                 final EditText input = new EditText(getActivity());
diff --git a/app/src/main/res/drawable/bg_delete_button.xml b/app/src/main/res/drawable/bg_delete_button.xml
index 8fef090ab..199940e1b 100644
--- a/app/src/main/res/drawable/bg_delete_button.xml
+++ b/app/src/main/res/drawable/bg_delete_button.xml
@@ -1,7 +1,32 @@
 <?xml version="1.0" encoding="utf-8"?>
-<shape xmlns:android="http://schemas.android.com/apk/res/android"
-    android:shape="rectangle">
-    <solid android:color="@color/deleteButtonColor"/>
-    <corners android:radius="3dp" />
-    <stroke android:width="5px" android:color="@color/deleteBackgroundColor" />
-</shape>
\ No newline at end of file
+<selector
+    android:layout_height="match_parent"
+    android:layout_width="match_parent"
+    xmlns:android="http://schemas.android.com/apk/res/android">
+
+    <item
+        android:state_enabled="true" >
+        <shape
+            android:shape="rectangle">
+            <solid
+                android:color="@color/deleteButton"/>
+            <corners
+                android:radius="3dp" />
+            <stroke
+                android:width="5px"
+                android:color="@color/deleteRed" />
+        </shape>
+    </item>
+
+    <item
+        android:state_enabled="false" >
+        <shape
+            android:shape="rectangle">
+            <solid
+                android:color="@color/deleteButtonDark"/>
+            <corners
+                android:radius="3dp" />
+        </shape>
+    </item>
+
+</selector>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_media_detail.xml b/app/src/main/res/layout/fragment_media_detail.xml
index 2b6ecf314..190eb011d 100644
--- a/app/src/main/res/layout/fragment_media_detail.xml
+++ b/app/src/main/res/layout/fragment_media_detail.xml
@@ -253,7 +253,7 @@
 
                 <LinearLayout
                     android:id="@+id/nominatedDeletionBanner"
-                    android:background="@color/deleteBackgroundColor"
+                    android:background="@color/deleteRed"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:orientation="vertical"
@@ -281,8 +281,8 @@
 
                 <Button
                     android:id="@+id/nominateDeletion"
-                    android:textColor="@color/primaryTextColor"
                     android:background="@drawable/bg_delete_button"
+                    android:textColor="@color/primaryTextColor"
                     android:layout_margin="@dimen/standard_gap"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index 4ce597181..c9e2b2764 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -15,8 +15,10 @@
     <color name="primaryTextColor">#ffffff</color>
     <color name="secondaryTextColor">#000000</color>
 
-    <color name="deleteBackgroundColor">#90960a0a</color>
-    <color name="deleteButtonColor">#44000000</color>
+    <color name="deleteRed">#90960a0a</color>
+    <color name="deleteButton">#44000000</color>
+    <color name="deleteButtonDark">#88000000</color>
+    <color name="deleteButtonLight">#44ffffff</color>
 
     <!-- Some colours are same for dark/light themes. They are written two times in case
     we want to change light ones later.

From 60f7e6aba8cad068882baf1f741d8cbddc3e60a6 Mon Sep 17 00:00:00 2001
From: Suchit Kar <suchitkar1997@gmail.com>
Date: Wed, 21 Mar 2018 17:38:41 +0530
Subject: [PATCH 4/5] Move try blocks to single block in DeleteTask

---
 .../free/nrw/commons/delete/DeleteTask.java   | 65 ++++++++-----------
 1 file changed, 28 insertions(+), 37 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
index c75c05e69..25f8ac2c2 100644
--- a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
+++ b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
@@ -23,7 +23,7 @@ import timber.log.Timber;
 import static android.support.v4.content.ContextCompat.startActivity;
 import static android.widget.Toast.LENGTH_SHORT;
 
-public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
+public class DeleteTask extends AsyncTask<Void, Integer, Boolean> {
 
     @Inject MediaWikiApi mwApi;
     @Inject SessionManager sessionManager;
@@ -55,17 +55,6 @@ public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
         authCookie = sessionManager.getAuthCookie();
         mwApi.setAuthCookie(authCookie);
 
-        try {
-            editToken = mwApi.getEditToken();
-        }
-        catch (Exception e){
-            Timber.d(e.getMessage());
-            return false;
-        }
-        if (editToken.equals("+\\")) {
-            return false;
-        }
-
         Calendar calendar = Calendar.getInstance();
         String fileDeleteString = "{{delete|reason=" + reason +
                 "|subpage=" +media.getFilename() +
@@ -73,43 +62,38 @@ public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
                 "|month=" + calendar.getDisplayName(Calendar.MONTH,Calendar.LONG, Locale.getDefault()) +
                 "|year=" + calendar.get(Calendar.YEAR) +
                 "}}";
-        try{
-            mwApi.prependEdit(editToken,fileDeleteString+"\n",
-                    media.getFilename(),summary);
-        }
-        catch (Exception e) {
-            Timber.d(e.getMessage());
-            return false;
-        }
 
         String subpageString = "=== [[:" + media.getFilename() + "]] ===\n" +
                 reason +
                 " ~~~~";
-        try{
-            mwApi.edit(editToken,subpageString+"\n",
-                    "Commons:Deletion_requests/"+media.getFilename(),summary);
-        }
-        catch (Exception e) {
-            Timber.d(e.getMessage());
-            return false;
-        }
 
         String logPageString = "\n{{Commons:Deletion requests/" + media.getFilename() +
                 "}}\n";
         SimpleDateFormat sdf = new SimpleDateFormat("yyyy/MM/dd");
         String date = sdf.format(calendar.getTime());
-        try{
-            mwApi.appendEdit(editToken,logPageString+"\n",
-                    "Commons:Deletion_requests/"+date,summary);
-        }
-        catch (Exception e) {
-            Timber.d(e.getMessage());
-            return false;
-        }
 
         String userPageString = "\n{{subst:idw|" + media.getFilename() +
                 "}} ~~~~";
-        try{
+
+        try {
+            editToken = mwApi.getEditToken();
+            if (editToken.equals("+\\")) {
+                return false;
+            }
+            publishProgress(1);
+
+            mwApi.prependEdit(editToken,fileDeleteString+"\n",
+                    media.getFilename(),summary);
+            publishProgress(2);
+
+            mwApi.edit(editToken,subpageString+"\n",
+                    "Commons:Deletion_requests/"+media.getFilename(),summary);
+            publishProgress(3);
+
+            mwApi.appendEdit(editToken,logPageString+"\n",
+                    "Commons:Deletion_requests/"+date,summary);
+            publishProgress(4);
+
             mwApi.appendEdit(editToken,userPageString+"\n",
                     "User_Talk:"+sessionManager.getCurrentAccount().name,summary);
         }
@@ -120,10 +104,16 @@ public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
         return true;
     }
 
+    @Override
+    protected void onProgressUpdate (Integer... values){
+
+    }
+
     @Override
     protected void onPostExecute(Boolean result) {
         String message = "";
         String title = "";
+
         if (result){
             title = "Success";
             message = "Successfully nominated " + media.getDisplayTitle() + " deletion.\n" +
@@ -133,6 +123,7 @@ public class DeleteTask extends AsyncTask<Void, Void, Boolean> {
             title = "Failed";
             message = "Could not request deletion. Something went wrong.";
         }
+
         AlertDialog alert;
         AlertDialog.Builder builder = new AlertDialog.Builder(context);
         builder.setTitle(title);

From 2a62fe43314f4c25f5fdf533ed1bdc54f302f426 Mon Sep 17 00:00:00 2001
From: Suchit Kar <suchitkar1997@gmail.com>
Date: Thu, 22 Mar 2018 22:11:17 +0530
Subject: [PATCH 5/5] Display deletion progress as notifications

---
 .../free/nrw/commons/delete/DeleteTask.java   | 99 ++++++++++++-------
 1 file changed, 64 insertions(+), 35 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
index 25f8ac2c2..0cce496f0 100644
--- a/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
+++ b/app/src/main/java/fr/free/nrw/commons/delete/DeleteTask.java
@@ -1,10 +1,11 @@
 package fr.free.nrw.commons.delete;
 
-import android.app.AlertDialog;
+import android.app.NotificationManager;
 import android.content.Context;
-import android.content.DialogInterface;
-import android.content.Intent;
 import android.os.AsyncTask;
+import android.support.v4.app.NotificationCompat;
+import android.support.v4.app.NotificationCompat.Builder;
+import android.view.Gravity;
 import android.widget.Toast;
 
 import java.text.SimpleDateFormat;
@@ -20,14 +21,18 @@ import fr.free.nrw.commons.di.ApplicationlessInjection;
 import fr.free.nrw.commons.mwapi.MediaWikiApi;
 import timber.log.Timber;
 
-import static android.support.v4.content.ContextCompat.startActivity;
-import static android.widget.Toast.LENGTH_SHORT;
+import static android.support.v4.app.NotificationCompat.DEFAULT_ALL;
+import static android.support.v4.app.NotificationCompat.PRIORITY_HIGH;
 
 public class DeleteTask extends AsyncTask<Void, Integer, Boolean> {
 
     @Inject MediaWikiApi mwApi;
     @Inject SessionManager sessionManager;
 
+    public static final int NOTIFICATION_DELETE = 1;
+
+    private NotificationManager notificationManager;
+    private Builder notificationBuilder;
     private Context context;
     private Media media;
     private String reason;
@@ -44,10 +49,19 @@ public class DeleteTask extends AsyncTask<Void, Integer, Boolean> {
                 .getInstance(context.getApplicationContext())
                 .getCommonsApplicationComponent()
                 .inject(this);
+
+        notificationManager = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);
+        notificationBuilder = new NotificationCompat.Builder(context);
+        Toast toast = new Toast(context);
+        toast.setGravity(Gravity.CENTER,0,0);
+        toast = Toast.makeText(context,"Trying to nominate "+media.getDisplayTitle()+ " for deletion",Toast.LENGTH_SHORT);
+        toast.show();
     }
 
     @Override
     protected Boolean doInBackground(Void ...voids) {
+        publishProgress(0);
+
         String editToken;
         String authCookie;
         String summary = "Nominating " + media.getFilename() +" for deletion.";
@@ -96,6 +110,7 @@ public class DeleteTask extends AsyncTask<Void, Integer, Boolean> {
 
             mwApi.appendEdit(editToken,userPageString+"\n",
                     "User_Talk:"+sessionManager.getCurrentAccount().name,summary);
+            publishProgress(5);
         }
         catch (Exception e) {
             Timber.d(e.getMessage());
@@ -106,47 +121,61 @@ public class DeleteTask extends AsyncTask<Void, Integer, Boolean> {
 
     @Override
     protected void onProgressUpdate (Integer... values){
+        super.onProgressUpdate(values);
 
+        String message = "";
+        switch (values[0]){
+            case 0:
+                message = "Getting token";
+                break;
+            case 1:
+                message = "Adding delete message to file";
+                break;
+            case 2:
+                message = "Creating Delete requests sub-page";
+                break;
+            case 3:
+                message = "Adding file to Delete requests log";
+                break;
+            case 4:
+                message = "Notifying User on Talk page";
+                break;
+            case 5:
+                message = "Done";
+                break;
+        }
+
+        notificationBuilder.setContentTitle("Nominating "+media.getDisplayTitle()+" for deletion")
+                .setStyle(new NotificationCompat.BigTextStyle()
+                        .bigText(message))
+                .setSmallIcon(R.drawable.ic_launcher)
+                .setProgress(5, values[0], false)
+                .setOngoing(true);
+        notificationManager.notify(NOTIFICATION_DELETE, notificationBuilder.build());
     }
 
     @Override
     protected void onPostExecute(Boolean result) {
         String message = "";
-        String title = "";
+        String title = "Nominating for Deletion";
 
         if (result){
-            title = "Success";
-            message = "Successfully nominated " + media.getDisplayTitle() + " deletion.\n" +
-                    "Check the webpage for more details";
+            title += ": Success";
+            message = "Successfully nominated " + media.getDisplayTitle() + " deletion.";
         }
         else {
-            title = "Failed";
-            message = "Could not request deletion. Something went wrong.";
+            title += ": Failed";
+            message = "Could not request deletion.";
         }
 
-        AlertDialog alert;
-        AlertDialog.Builder builder = new AlertDialog.Builder(context);
-        builder.setTitle(title);
-        builder.setMessage(message);
-        builder.setCancelable(true);
-        builder.setPositiveButton(
-                R.string.ok,
-                new DialogInterface.OnClickListener() {
-                    public void onClick(DialogInterface dialog, int id) {}
-                });
-        builder.setNeutralButton(R.string.view_browser,
-                new DialogInterface.OnClickListener() {
-                    public void onClick(DialogInterface dialog, int id) {
-                        Intent browserIntent = new Intent(Intent.ACTION_VIEW, media.getFilePageTitle().getMobileUri());
-                        if (browserIntent.resolveActivity(context.getPackageManager()) != null) {
-                            startActivity(context,browserIntent,null);
-                        } else {
-                            Toast toast = Toast.makeText(context, R.string.no_web_browser, LENGTH_SHORT);
-                            toast.show();
-                        }
-                    }
-                });
-        alert = builder.create();
-        alert.show();
+        notificationBuilder.setDefaults(DEFAULT_ALL)
+                .setContentTitle(title)
+                .setStyle(new NotificationCompat.BigTextStyle()
+                        .bigText(message))
+                .setSmallIcon(R.drawable.ic_launcher)
+                .setProgress(0,0,false)
+                .setOngoing(false)
+                .setPriority(PRIORITY_HIGH);
+        notificationManager.notify(NOTIFICATION_DELETE, notificationBuilder.build());
     }
 }
\ No newline at end of file