From c1fc6959e797958cbc8f8028ddf31158b967e93b Mon Sep 17 00:00:00 2001
From: Eugen Rochko <eugen@zeonfederated.com>
Date: Fri, 10 Mar 2017 23:47:04 +0100
Subject: [PATCH] Fix up report activity/layout

---
 .../keylesspalace/tusky/ReportActivity.java   | 73 ++++++++++++-----
 app/src/main/res/layout/activity_report.xml   | 80 +++++++++----------
 app/src/main/res/menu/report_toolbar.xml      | 10 +++
 app/src/main/res/values/strings.xml           |  2 +-
 app/src/main/res/values/styles.xml            |  4 +-
 5 files changed, 101 insertions(+), 68 deletions(-)
 create mode 100644 app/src/main/res/menu/report_toolbar.xml

diff --git a/app/src/main/java/com/keylesspalace/tusky/ReportActivity.java b/app/src/main/java/com/keylesspalace/tusky/ReportActivity.java
index e53601c5..3f4b5c0b 100644
--- a/app/src/main/java/com/keylesspalace/tusky/ReportActivity.java
+++ b/app/src/main/java/com/keylesspalace/tusky/ReportActivity.java
@@ -17,6 +17,7 @@ package com.keylesspalace.tusky;
 
 import android.content.Intent;
 import android.graphics.drawable.Drawable;
+import android.net.Uri;
 import android.os.Bundle;
 import android.support.annotation.Nullable;
 import android.support.design.widget.Snackbar;
@@ -25,6 +26,8 @@ import android.support.v7.widget.DividerItemDecoration;
 import android.support.v7.widget.LinearLayoutManager;
 import android.support.v7.widget.RecyclerView;
 import android.support.v7.widget.Toolbar;
+import android.view.Menu;
+import android.view.MenuItem;
 import android.view.View;
 import android.widget.Button;
 import android.widget.EditText;
@@ -45,6 +48,11 @@ public class ReportActivity extends BaseActivity {
     private View anyView; // what Snackbar will use to find the root view
     private ReportAdapter adapter;
     private boolean reportAlreadyInFlight;
+    private String accountId;
+    private String accountUsername;
+    private String statusId;
+    private String statusContent;
+    private EditText comment;
 
     @Override
     protected void onCreate(@Nullable Bundle savedInstanceState) {
@@ -52,10 +60,10 @@ public class ReportActivity extends BaseActivity {
         setContentView(R.layout.activity_report);
 
         Intent intent = getIntent();
-        final String accountId = intent.getStringExtra("account_id");
-        String accountUsername = intent.getStringExtra("account_username");
-        String statusId = intent.getStringExtra("status_id");
-        String statusContent = intent.getStringExtra("status_content");
+        accountId = intent.getStringExtra("account_id");
+        accountUsername = intent.getStringExtra("account_username");
+        statusId = intent.getStringExtra("status_id");
+        statusContent = intent.getStringExtra("status_content");
 
         Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
         setSupportActionBar(toolbar);
@@ -64,6 +72,8 @@ public class ReportActivity extends BaseActivity {
             String title = String.format(getString(R.string.report_username_format),
                     accountUsername);
             bar.setTitle(title);
+            bar.setDisplayHomeAsUpEnabled(true);
+            bar.setDisplayShowHomeEnabled(true);
         }
         anyView = toolbar;
 
@@ -85,28 +95,28 @@ public class ReportActivity extends BaseActivity {
                 HtmlUtils.fromHtml(statusContent), true);
         adapter.addItem(reportStatus);
 
-        final EditText comment = (EditText) findViewById(R.id.report_comment);
-        Button send = (Button) findViewById(R.id.report_send);
+        comment = (EditText) findViewById(R.id.report_comment);
+
         reportAlreadyInFlight = false;
-        send.setOnClickListener(new View.OnClickListener() {
-            @Override
-            public void onClick(View v) {
-                if (reportAlreadyInFlight) {
-                    return;
-                }
-                String[] statusIds = adapter.getCheckedStatusIds();
-                if (statusIds.length > 0) {
-                    reportAlreadyInFlight = true;
-                    sendReport(accountId, statusIds, comment.getText().toString());
-                } else {
-                    comment.setError(getString(R.string.error_report_too_few_statuses));
-                }
-            }
-        });
 
         fetchRecentStatuses(accountId);
     }
 
+    private void onClickSend() {
+        if (reportAlreadyInFlight) {
+            return;
+        }
+
+        String[] statusIds = adapter.getCheckedStatusIds();
+
+        if (statusIds.length > 0) {
+            reportAlreadyInFlight = true;
+            sendReport(accountId, statusIds, comment.getText().toString());
+        } else {
+            comment.setError(getString(R.string.error_report_too_few_statuses));
+        }
+    }
+
     private void sendReport(final String accountId, final String[] statusIds,
             final String comment) {
         mastodonAPI.report(accountId, Arrays.asList(statusIds), comment).enqueue(new Callback<ResponseBody>() {
@@ -167,4 +177,25 @@ public class ReportActivity extends BaseActivity {
     private void onFetchStatusesFailure(Exception exception) {
         Log.e(TAG, "Failed to fetch recent statuses to report. " + exception.getMessage());
     }
+
+    @Override
+    public boolean onCreateOptionsMenu(Menu menu) {
+        getMenuInflater().inflate(R.menu.report_toolbar, menu);
+        return super.onCreateOptionsMenu(menu);
+    }
+
+    @Override
+    public boolean onOptionsItemSelected(MenuItem item) {
+        switch (item.getItemId()) {
+            case android.R.id.home: {
+                onBackPressed();
+                return true;
+            }
+            case R.id.action_report: {
+                onClickSend();
+                return true;
+            }
+        }
+        return super.onOptionsItemSelected(item);
+    }
 }
diff --git a/app/src/main/res/layout/activity_report.xml b/app/src/main/res/layout/activity_report.xml
index 9b884f32..c9f89a22 100644
--- a/app/src/main/res/layout/activity_report.xml
+++ b/app/src/main/res/layout/activity_report.xml
@@ -1,51 +1,43 @@
 <?xml version="1.0" encoding="utf-8"?>
-<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<android.support.design.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:orientation="vertical"
     android:layout_width="match_parent"
     android:layout_height="match_parent">
-
-    <android.support.v7.widget.Toolbar
-        android:id="@+id/toolbar"
+    <LinearLayout
         android:layout_width="match_parent"
-        android:layout_height="?attr/actionBarSize"
-        android:elevation="4dp"
-        android:theme="@style/AppTheme.Account.AppBarLayout"
-        app:popupTheme="@style/AppTheme.Account.ToolbarPopupTheme.Dark"
-        android:background="?attr/toolbar_background_color" />
+        android:layout_height="match_parent"
+        android:orientation="vertical">
+        <android.support.v7.widget.Toolbar
+            android:id="@+id/toolbar"
+            android:layout_width="match_parent"
+            android:layout_height="?attr/actionBarSize"
+            android:elevation="4dp"
+            android:theme="@style/AppTheme.Account.AppBarLayout"
+            app:popupTheme="@style/AppTheme.Account.ToolbarPopupTheme.Dark"
+            android:background="?attr/toolbar_background_color" />
 
-    <android.support.v7.widget.RecyclerView
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
-        android:layout_weight="2"
-        android:id="@+id/report_recycler_view"
-        android:scrollbars="vertical"
-        android:fadeScrollbars="false"
-        android:background="?attr/report_status_background_color" />
+        <android.support.v7.widget.RecyclerView
+            android:layout_width="match_parent"
+            android:layout_height="0dp"
+            android:layout_weight="2"
+            android:id="@+id/report_recycler_view"
+            android:scrollbars="vertical"
+            android:fadeScrollbars="false"
+            android:background="?attr/report_status_background_color" />
 
-    <EditText
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
-        android:layout_weight="1"
-        android:id="@+id/report_comment"
-        android:inputType="textMultiLine"
-        android:gravity="top|start"
-        android:ems="10"
-        android:layout_marginLeft="8dp"
-        android:layout_marginRight="8dp"
-        android:hint="@string/report_comment_hint" />
-
-    <RelativeLayout
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content">
-
-        <Button
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_alignParentRight="true"
-            android:id="@+id/report_send"
-            android:text="@string/action_report" />
-
-    </RelativeLayout>
-
-</LinearLayout>
\ No newline at end of file
+        <EditText
+            android:layout_width="match_parent"
+            android:layout_height="0dp"
+            android:layout_weight="1"
+            android:id="@+id/report_comment"
+            android:inputType="textMultiLine"
+            android:gravity="top|start"
+            android:background="@android:color/transparent"
+            android:ems="10"
+            android:paddingLeft="16dp"
+            android:paddingRight="16dp"
+            android:paddingTop="8dp"
+            android:paddingBottom="8dp"
+            android:hint="@string/report_comment_hint" />
+    </LinearLayout>
+</android.support.design.widget.CoordinatorLayout>
\ No newline at end of file
diff --git a/app/src/main/res/menu/report_toolbar.xml b/app/src/main/res/menu/report_toolbar.xml
new file mode 100644
index 00000000..d3b3ee8f
--- /dev/null
+++ b/app/src/main/res/menu/report_toolbar.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8"?>
+<menu xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:android="http://schemas.android.com/apk/res/android">
+
+    <item
+        android:id="@+id/action_report"
+        android:title="@string/action_report"
+        android:icon="@drawable/ic_send_24dp"
+        app:showAsAction="always"/>
+</menu>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 558e3593..006f4999 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -54,7 +54,7 @@
     <string name="notification_favourite_format">%s favourited your status</string>
     <string name="notification_follow_format">%s followed you</string>
 
-    <string name="report_username_format">Reporting @%s</string>
+    <string name="report_username_format">Report @%s</string>
     <string name="report_comment_hint">Additional comments?</string>
 
     <string name="action_compose">Compose</string>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 5ffc65e2..4e651e20 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -53,8 +53,8 @@
         <item name="compose_content_warning_bar_background">@drawable/border_background_dark</item>
         <item name="notification_content">@color/notification_content_faded_dark</item>
         <item name="notification_icon_tint">@color/notification_icon_tint_dark</item>
-        <item name="report_status_background_color">@color/report_status_background_dark</item>
-        <item name="report_status_divider_drawable">@drawable/report_status_divider_dark</item>
+        <item name="report_status_background_color">@color/color_background_dark</item>
+        <item name="report_status_divider_drawable">@drawable/status_divider_dark</item>
 
         <item name="material_drawer_background">@color/window_background_dark</item>
         <item name="material_drawer_primary_text">@color/text_color_primary_dark</item>