From b5f5259366a1af911f69eba88b458abf333fe993 Mon Sep 17 00:00:00 2001
From: Sam Bosley <sboz88@gmail.com>
Date: Tue, 24 Apr 2012 14:04:28 -0700
Subject: [PATCH] Fixed a bug with dialog theme declared in manifest messing up
 tag settings activity on phones

---
 astrid/AndroidManifest.xml                               | 2 ++
 .../todoroo/astrid/actfm/TagSettingsActivityTablet.java  | 9 +++++++++
 .../com/todoroo/astrid/actfm/TagViewFragment.java        | 8 +++++---
 .../plugin-src/com/todoroo/astrid/tags/TagsPlugin.java   | 5 ++++-
 4 files changed, 20 insertions(+), 4 deletions(-)
 create mode 100644 astrid/plugin-src/com/todoroo/astrid/actfm/TagSettingsActivityTablet.java

diff --git a/astrid/AndroidManifest.xml b/astrid/AndroidManifest.xml
index 0708e7888..f21d2e1ac 100644
--- a/astrid/AndroidManifest.xml
+++ b/astrid/AndroidManifest.xml
@@ -355,6 +355,8 @@
     <activity android:name="com.todoroo.astrid.actfm.TagCreateActivity"
               android:theme="@android:style/Theme.Dialog"/>
     <activity android:name="com.todoroo.astrid.actfm.TagSettingsActivity"
+              android:windowSoftInputMode="stateHidden"/>
+    <activity android:name="com.todoroo.astrid.actfm.TagSettingsActivityTablet"
               android:windowSoftInputMode="stateHidden"
               android:theme="@style/Theme.Dialog"/>
     <activity android:name="com.todoroo.astrid.actfm.TagUpdatesActivity"
diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/TagSettingsActivityTablet.java b/astrid/plugin-src/com/todoroo/astrid/actfm/TagSettingsActivityTablet.java
new file mode 100644
index 000000000..fc84a503f
--- /dev/null
+++ b/astrid/plugin-src/com/todoroo/astrid/actfm/TagSettingsActivityTablet.java
@@ -0,0 +1,9 @@
+package com.todoroo.astrid.actfm;
+
+public class TagSettingsActivityTablet extends TagSettingsActivity {
+    /**
+     * This class exists for the sole purpose of being declared separately in the manifest
+     * so that we don't get crazy bugs with it being Dialog Theme on tablets and normal theme
+     * on phones.
+     */
+}
diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/TagViewFragment.java b/astrid/plugin-src/com/todoroo/astrid/actfm/TagViewFragment.java
index 2814401c7..2dd1ee450 100644
--- a/astrid/plugin-src/com/todoroo/astrid/actfm/TagViewFragment.java
+++ b/astrid/plugin-src/com/todoroo/astrid/actfm/TagViewFragment.java
@@ -125,11 +125,13 @@ public class TagViewFragment extends TaskListFragment {
     private final OnClickListener settingsListener = new OnClickListener() {
         @Override
         public void onClick(View v) {
-            Intent intent = new Intent(getActivity(), TagSettingsActivity.class);
+            Activity activity = getActivity();
+            Class<?> settingsClass = AndroidUtilities.isTabletSized(activity) ? TagSettingsActivityTablet.class : TagSettingsActivity.class;
+            Intent intent = new Intent(getActivity(), settingsClass);
             intent.putExtra(EXTRA_TAG_DATA, tagData);
             startActivityForResult(intent, REQUEST_CODE_SETTINGS);
-            if (!AndroidUtilities.isTabletSized(getActivity())) {
-                AndroidUtilities.callOverridePendingTransition(getActivity(), R.anim.slide_left_in, R.anim.slide_left_out);
+            if (!AndroidUtilities.isTabletSized(activity)) {
+                AndroidUtilities.callOverridePendingTransition(activity, R.anim.slide_left_in, R.anim.slide_left_out);
             }
         }
     };
diff --git a/astrid/plugin-src/com/todoroo/astrid/tags/TagsPlugin.java b/astrid/plugin-src/com/todoroo/astrid/tags/TagsPlugin.java
index 32d19a880..7f185225c 100644
--- a/astrid/plugin-src/com/todoroo/astrid/tags/TagsPlugin.java
+++ b/astrid/plugin-src/com/todoroo/astrid/tags/TagsPlugin.java
@@ -4,7 +4,9 @@ import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
 
+import com.todoroo.andlib.utility.AndroidUtilities;
 import com.todoroo.astrid.actfm.TagSettingsActivity;
+import com.todoroo.astrid.actfm.TagSettingsActivityTablet;
 import com.todoroo.astrid.api.Addon;
 import com.todoroo.astrid.api.AstridApiConstants;
 
@@ -29,7 +31,8 @@ public class TagsPlugin extends BroadcastReceiver {
      * @param activity
      */
     public static Intent newTagDialog(Context context) {
-        Intent intent = new Intent(context, TagSettingsActivity.class);
+        Class<?> settingsComponent = AndroidUtilities.isTabletSized(context) ? TagSettingsActivityTablet.class : TagSettingsActivity.class;
+        Intent intent = new Intent(context, settingsComponent);
         return intent;
     }