Merge pull request commons-app#638 from maskaravivek/shareBack

Fix back button issue in ShareActivity

Author: misaochan

app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java

@@ -498,7 +498,7 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
         }
     }
 
-    private void backButtonDialog() {
+    public void backButtonDialog() {
         AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
 
         builder.setMessage("Are you sure you want to go back? The image will not have any categories saved.")

app/src/main/java/fr/free/nrw/commons/contributions/ContributionsActivity.java

@@ -138,6 +138,7 @@ protected void onCreate(Bundle savedInstanceState) {
         }
         requestAuthToken();
         initDrawer();
+        setTitle(getString(R.string.title_activity_contributions));
     }
 
     @Override

app/src/main/java/fr/free/nrw/commons/theme/NavigationBaseActivity.java

@@ -67,6 +67,12 @@ public void onClick(View v) {
         });
     }
 
+    public void initBack() {
+        setSupportActionBar(toolbar);
+        getSupportActionBar().setDisplayHomeAsUpEnabled(true);
+        getSupportActionBar().setDisplayShowHomeEnabled(true);
+    }
+
     private void setDrawerPaneWidth() {
         ViewGroup.LayoutParams params = drawerPane.getLayoutParams();
         // set width to lowerBound of 80% of the screen size

app/src/main/java/fr/free/nrw/commons/upload/ShareActivity.java

@@ -219,7 +219,7 @@ public void onCreate(Bundle savedInstanceState) {
         uploadController = new UploadController(this);
         setContentView(R.layout.activity_share);
         ButterKnife.bind(this);
-        initDrawer();
+        initBack();
         app = CommonsApplication.getInstance();
         backgroundImageView = (SimpleDraweeView)findViewById(R.id.backgroundImage);
 
@@ -444,7 +444,11 @@ protected void onDestroy() {
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
             case android.R.id.home:
-                NavUtils.navigateUpFromSameTask(this);
+                if(categorizationFragment!=null && categorizationFragment.isVisible()) {
+                    categorizationFragment.backButtonDialog();
+                } else {
+                    onBackPressed();
+                }
                 return true;
         }
         return super.onOptionsItemSelected(item);