From a4e0242672909835c24939daf2bd7da5b95bdef8 Mon Sep 17 00:00:00 2001
From: kbhardwaj123 <kshitij256kencryption@gmail.com>
Date: Thu, 13 Feb 2020 03:22:31 +0530
Subject: [PATCH 1/3] ic_custom_greyed_out_marker: removing grey marker

---
 .../res/drawable/ic_custom_greyed_out_marker.xml     | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/src/main/res/drawable/ic_custom_greyed_out_marker.xml b/app/src/main/res/drawable/ic_custom_greyed_out_marker.xml
index 70521cbe93..e57db5f0cb 100644
--- a/app/src/main/res/drawable/ic_custom_greyed_out_marker.xml
+++ b/app/src/main/res/drawable/ic_custom_greyed_out_marker.xml
@@ -4,20 +4,20 @@
     android:viewportWidth="24.0"
     android:viewportHeight="28.0">
     <path
-        android:fillAlpha="0.1"
+        android:fillAlpha="0.01"
         android:fillColor="#48000000"
         android:pathData="M6.072,22.223a6.031,3.672 0,1 0,12.062 0a6.031,3.672 0,1 0,-12.062 0z"
         android:strokeWidth="1" />
     <path
-        android:fillAlpha="1"
-        android:fillColor="#4D000000"
+        android:fillAlpha="0.01"
+        android:fillColor="#48000000"
         android:pathData="M11.575,11.62C10.689,11.462 9.902,10.759 9.625,9.878 9.553,9.65 9.535,9.499 9.538,9.14c0.004,-0.397 0.019,-0.492 0.13,-0.787 0.236,-0.631 0.646,-1.099 1.212,-1.382 0.386,-0.193 0.709,-0.272 1.116,-0.272 0.676,0 1.263,0.247 1.744,0.734 0.355,0.359 0.541,0.682 0.657,1.136 0.327,1.278 -0.442,2.611 -1.723,2.987 -0.282,0.083 -0.817,0.114 -1.099,0.063z"
         android:strokeWidth="1" />
     <path
-        android:fillAlpha="1"
-        android:fillColor="#7CE4E0D9"
+        android:fillAlpha="0.01"
+        android:fillColor="#48000000"
         android:pathData="M11.617,21.707C10.518,20.424 9.338,18.864 8.395,17.449 6.524,14.641 5.455,12.305 5.102,10.255 5.014,9.744 5.006,8.628 5.088,8.137 5.348,6.561 6.043,5.221 7.158,4.148 9.148,2.231 12.016,1.668 14.593,2.688c2.043,0.809 3.607,2.581 4.162,4.719 0.174,0.67 0.204,0.933 0.203,1.761 -0.001,0.81 -0.035,1.098 -0.22,1.857 -0.614,2.524 -2.571,5.977 -5.383,9.501 -0.645,0.809 -1.321,1.61 -1.358,1.61 -0.008,0 -0.179,-0.193 -0.381,-0.428zM12.617,11.603c0.783,-0.188 1.457,-0.795 1.738,-1.564 0.516,-1.415 -0.317,-2.962 -1.783,-3.312 -0.216,-0.052 -0.317,-0.059 -0.661,-0.047 -0.354,0.012 -0.441,0.025 -0.682,0.104 -0.673,0.221 -1.205,0.695 -1.506,1.344 -0.176,0.38 -0.218,0.584 -0.217,1.054 0.001,0.324 0.014,0.452 0.064,0.635 0.266,0.97 1.077,1.689 2.079,1.844 0.243,0.038 0.68,0.012 0.968,-0.057z"
         android:strokeAlpha="1"
         android:strokeColor="#54003B59"
-        android:strokeWidth="1" />
+        android:strokeWidth="0.001" />
 </vector>

From 339b4948c53930ba55b1cb03e4a3c3f664495371 Mon Sep 17 00:00:00 2001
From: kbhardwaj123 <kshitij256kencryption@gmail.com>
Date: Thu, 13 Feb 2020 03:23:41 +0530
Subject: [PATCH 2/3] NearbyParentFragment: changing function name and
 description

---
 .../nearby/fragments/NearbyParentFragment.java    | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java b/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
index e88be26260..d9145d6363 100644
--- a/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
@@ -1050,7 +1050,7 @@ public void updateMapMarkers(List<NearbyBaseMarker> nearbyBaseMarkers, Marker se
 
     @Override
     public void filterOutAllMarkers() {
-        greyOutAllMarkers();
+        removeAllMarkers();
     }
 
     /**
@@ -1077,7 +1077,8 @@ public void filterMarkersByLabels(List<Label> selectedLabels, boolean displayExi
                                       boolean filterForPlaceState,
                                       boolean filterForAllNoneType) {
         if (selectedLabels.size() == 0 && filterForPlaceState) { // If nothing is selected, display all
-            greyOutAllMarkers();
+            // remove the previous markers before updating them
+            removeAllMarkers();
             for (MarkerPlaceGroup markerPlaceGroup : NearbyController.markerLabelList) {
                 if (displayExists && displayNeedsPhoto) {
                     // Exists and needs photo
@@ -1101,8 +1102,8 @@ public void filterMarkersByLabels(List<Label> selectedLabels, boolean displayExi
 
             }
         } else {
-            // First greyed out all markers
-            greyOutAllMarkers();
+            // First remove all the markers
+            removeAllMarkers();
             for (MarkerPlaceGroup markerPlaceGroup : NearbyController.markerLabelList) {
                 for (Label label : selectedLabels) {
                     if (markerPlaceGroup.getPlace().getLabel().toString().equals(label.toString())) {
@@ -1186,9 +1187,11 @@ public void updateMarker(boolean isBookmarked, Place place, @Nullable fr.free.nr
     }
 
     /**
-     * Greys out all markers except current location marker
+     * Removes all markers except current location marker, an icon has been used
+     * but it is transparent more than grey(as the name of the icon might suggest)
+     * since grey icon may lead the users to believe that it is disabled or prohibited contribution
      */
-    private void greyOutAllMarkers() {
+    private void removeAllMarkers() {
         if(currentLocationMarker==null){
             return;
         }

From 88183d7a9b044c45907f399f33c120e849dfe18d Mon Sep 17 00:00:00 2001
From: kbhardwaj123 <kshitij256kencryption@gmail.com>
Date: Thu, 13 Feb 2020 12:21:02 +0530
Subject: [PATCH 3/3] change method name to hideAllMarkers

---
 .../commons/nearby/fragments/NearbyParentFragment.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java b/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
index d9145d6363..c823fc05a9 100644
--- a/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/nearby/fragments/NearbyParentFragment.java
@@ -1050,7 +1050,7 @@ public void updateMapMarkers(List<NearbyBaseMarker> nearbyBaseMarkers, Marker se
 
     @Override
     public void filterOutAllMarkers() {
-        removeAllMarkers();
+        hideAllMArkers();
     }
 
     /**
@@ -1078,7 +1078,7 @@ public void filterMarkersByLabels(List<Label> selectedLabels, boolean displayExi
                                       boolean filterForAllNoneType) {
         if (selectedLabels.size() == 0 && filterForPlaceState) { // If nothing is selected, display all
             // remove the previous markers before updating them
-            removeAllMarkers();
+            hideAllMArkers();
             for (MarkerPlaceGroup markerPlaceGroup : NearbyController.markerLabelList) {
                 if (displayExists && displayNeedsPhoto) {
                     // Exists and needs photo
@@ -1103,7 +1103,7 @@ public void filterMarkersByLabels(List<Label> selectedLabels, boolean displayExi
             }
         } else {
             // First remove all the markers
-            removeAllMarkers();
+            hideAllMArkers();
             for (MarkerPlaceGroup markerPlaceGroup : NearbyController.markerLabelList) {
                 for (Label label : selectedLabels) {
                     if (markerPlaceGroup.getPlace().getLabel().toString().equals(label.toString())) {
@@ -1191,7 +1191,7 @@ public void updateMarker(boolean isBookmarked, Place place, @Nullable fr.free.nr
      * but it is transparent more than grey(as the name of the icon might suggest)
      * since grey icon may lead the users to believe that it is disabled or prohibited contribution
      */
-    private void removeAllMarkers() {
+    private void hideAllMArkers() {
         if(currentLocationMarker==null){
             return;
         }