From 2e72bb936a7773882f29ab1ebd182e6c37d1af37 Mon Sep 17 00:00:00 2001
From: Sam Bosley <sboz88@gmail.com>
Date: Mon, 16 Jan 2012 14:36:03 -0800
Subject: [PATCH] Adjusted parent style of Theme.Dialog to be
 android:Theme.Dialog to fix a crash in EditNoteActivity

---
 astrid/AndroidManifest.xml                    |  4 +-
 astrid/res/layout/task_edit_activity.xml      | 62 +------------------
 astrid/res/values/strings-core.xml            |  3 +
 astrid/res/values/styles.xml                  | 21 ++++++-
 .../astrid/activity/TaskEditActivity.java     | 35 +++++------
 5 files changed, 40 insertions(+), 85 deletions(-)

diff --git a/astrid/AndroidManifest.xml b/astrid/AndroidManifest.xml
index 5567a656e..0ae2b7074 100644
--- a/astrid/AndroidManifest.xml
+++ b/astrid/AndroidManifest.xml
@@ -450,7 +450,7 @@
     <!-- backup -->
     <service android:name="com.todoroo.astrid.backup.BackupService"/>
     <activity android:name="com.todoroo.astrid.backup.BackupActivity"
-              android:theme="@style/Theme.Dialog" />
+              android:theme="@style/Theme_Dialog" />
     <activity android:name="com.todoroo.astrid.backup.BackupPreferences"
               android:theme="@android:style/Theme" 
               android:label="@string/backup_BPr_header">
@@ -489,7 +489,7 @@
       </intent-filter>
     </receiver>
     <activity android:name="com.todoroo.astrid.notes.EditNoteActivity" 
-              android:theme="@style/Theme.Dialog"/>
+              android:theme="@style/Theme_Dialog"/>
     <!-- <receiver android:name="com.todoroo.astrid.notes.EditNoteExposer">
       <intent-filter>
         <action android:name="com.todoroo.astrid.REQUEST_ACTIONS" />
diff --git a/astrid/res/layout/task_edit_activity.xml b/astrid/res/layout/task_edit_activity.xml
index 3cc573f81..6b49ddac9 100644
--- a/astrid/res/layout/task_edit_activity.xml
+++ b/astrid/res/layout/task_edit_activity.xml
@@ -7,66 +7,6 @@
   android:layout_height="fill_parent"
   style="@style/Content">
   
-  	<!-- Header -->
-    <LinearLayout 
-        android:id="@+id/edit_header"
-	    android:layout_width="fill_parent" 
-	    android:layout_height="43dip"
-	    android:layout_weight="1"
-	    android:orientation="horizontal"
-	    android:layout_alignParentTop="true">
-
-	    <LinearLayout
-	        style="@style/Header"
-	        android:layout_width="fill_parent" 
-	        android:layout_height="fill_parent"
-	        android:layout_weight="1"
-	        android:orientation="horizontal">
-	    
-	        
-	        <!-- Header Logo -->
-	        <ImageView android:id="@+id/headerLogo"
-	            android:layout_width="fill_parent"
-	            android:layout_height="fill_parent"
-	            android:layout_weight="100"
-	            android:gravity="center"
-	            android:paddingRight="43dip"
-	            android:src="@drawable/header_logo_new"
-	            android:scaleType="center"
-	            android:visibility="gone"/>
-	           
-	        <!-- List Label -->
-	        <TextView android:id="@+id/taskLabel"
-	            android:layout_width="fill_parent"
-	            android:layout_height="fill_parent"
-	            android:layout_weight="100"
-	            android:singleLine="true"
-	            android:ellipsize="end"
-	            android:gravity="center"
-	            style="@style/TextAppearance.TLA_Header"/>    
-	            
-	    </LinearLayout>
-	   
-	   <LinearLayout android:id="@+id/activityContainer"
-           android:layout_width="wrap_content"
-           android:layout_height="fill_parent"
-           android:visibility="gone">
-            <View 
-               android:layout_width="0.5dip"
-               android:layout_height="fill_parent"
-               android:background="#ff222222"/>
-               
-            <ImageView android:id="@+id/activity"
-                android:layout_height="fill_parent"
-                android:layout_width="43dip"
-                android:background="?attr/asHeaderButtonBackground"
-                android:padding="7dip"
-                android:src="@drawable/icn_cmmt_off"
-                android:scaleType="fitCenter" /> 
-        </LinearLayout>
-	    
-    </LinearLayout>
-  
     <!-- buttons -->
 	<LinearLayout android:id="@+id/edit_footer"
 	  android:orientation="horizontal"
@@ -102,7 +42,7 @@
         
 	<ScrollView
 	  android:id="@+id/edit_scroll"
-	  android:layout_below="@id/edit_header"
+	  android:layout_alignParentTop="true"
 	  android:layout_above="@id/edit_footer"
 	  android:layout_width="fill_parent"
 	  android:layout_height="fill_parent"
diff --git a/astrid/res/values/strings-core.xml b/astrid/res/values/strings-core.xml
index d553783b5..22c6ac3ec 100644
--- a/astrid/res/values/strings-core.xml
+++ b/astrid/res/values/strings-core.xml
@@ -322,6 +322,9 @@
   <!-- Menu: Delete Task -->
   <string name="TEA_menu_delete">Delete Task</string>
   
+  <!-- Menu: Task comments -->
+  <string name="TEA_menu_comments">Comments</string>
+  
    <!-- Toast: task saved with deadline (%s => preposition + time units) -->
   <string name="TEA_onTaskSave_due">Task Saved: due %s</string>
   
diff --git a/astrid/res/values/styles.xml b/astrid/res/values/styles.xml
index ccc3bcc04..170276fee 100644
--- a/astrid/res/values/styles.xml
+++ b/astrid/res/values/styles.xml
@@ -75,7 +75,26 @@
         <item name="abLogo">@drawable/icon_48</item>
     </style>
     
-    <style name="Theme.Dialog">
+    <style name="Theme_Dialog" parent="@android:style/Theme.Dialog">
+        <item name="android:windowBackground">@null</item>
+        <item name="asContentBackground">@drawable/background_gradient</item>
+        <item name="asHeaderBackground">@drawable/header_background</item>
+        <item name="asHeaderButtonBackground">@drawable/header_button</item>
+        <item name="asSeparatorBackground">@android:drawable/divider_horizontal_dark</item>
+        <item name="asTextColor">#ffffffff</item>
+        <item name="asDetailsColor">#ff777777</item>
+        <item name="asDueDateColor">#ff7777aa</item>
+        <item name="asDueDateOverdueColor">#ffee5555</item>
+        <item name="asDueDateCompletedColor">#ff777777</item>
+        <item name="asFilterHeaderColor">#ffcccccc</item>
+        <item name="asFilterHeaderBackground">@drawable/edit_titlebar</item>
+        <item name="asMembersHeaderBackground">@drawable/ios_fabric_480_dark</item>
+        <item name="asSeparatorBackground">#888888</item>
+        <!-- actionbar-styling -->
+        <item name="abBackground">@drawable/header_background</item>
+        <item name="abDisplayOptions">useLogo|showHome|showTitle</item>
+        <item name="abLogo">@drawable/icon_48</item>
+        
         <item name="android:windowIsFloating">true</item>
         <item name="android:windowBackground">@drawable/panel_background</item>
         <item name="android:windowContentOverlay">@null</item>
diff --git a/astrid/src/com/todoroo/astrid/activity/TaskEditActivity.java b/astrid/src/com/todoroo/astrid/activity/TaskEditActivity.java
index 363a30fcc..cc2d12d5e 100755
--- a/astrid/src/com/todoroo/astrid/activity/TaskEditActivity.java
+++ b/astrid/src/com/todoroo/astrid/activity/TaskEditActivity.java
@@ -54,7 +54,6 @@ import android.widget.Button;
 import android.widget.EditText;
 import android.widget.ImageButton;
 import android.widget.LinearLayout;
-import android.widget.TextView;
 import android.widget.Toast;
 
 import com.timsu.astrid.R;
@@ -141,6 +140,7 @@ public final class TaskEditActivity extends Fragment {
     private static final int MENU_SAVE_ID = R.string.TEA_menu_save;
     private static final int MENU_DISCARD_ID = R.string.TEA_menu_discard;
     private static final int MENU_DELETE_ID = R.string.TEA_menu_delete;
+    private static final int MENU_COMMENTS_ID = R.string.TEA_menu_comments;
 
     // --- result codes
 
@@ -500,14 +500,6 @@ public final class TaskEditActivity extends Fragment {
 
             getView().findViewById(R.id.more_header).setOnClickListener(mExpandMoreListener);
 
-            getView().findViewById(R.id.activity).setOnClickListener(new View.OnClickListener() {
-                @Override
-                public void onClick(View v) {
-                    Intent launchIntent = new Intent(getActivity(), EditNoteActivity.class);
-                    launchIntent.putExtra(EditNoteActivity.EXTRA_TASK_ID, model.getId());
-                    startActivity(launchIntent);
-                }
-            });
         } catch (Exception e) {
             // error loading the proper activity
         }
@@ -644,12 +636,6 @@ public final class TaskEditActivity extends Fragment {
             return;
         }
 
-        if (!isNewTask) {
-            if (actFmPreferenceService.isLoggedIn()) {
-                getView().findViewById(R.id.activityContainer).setVisibility(View.VISIBLE);
-            }
-        }
-
         // clear notification
         Notifications.cancelNotifications(model.getId());
     }
@@ -664,12 +650,6 @@ public final class TaskEditActivity extends Fragment {
     public void populateFields(Intent intent) {
         loadItem(intent);
 
-        TextView titleText = (TextView) getView().findViewById(R.id.taskLabel);
-        if(isNewTask)
-            titleText.setText(R.string.TEA_view_titleNew);
-        else
-            titleText.setText(model.getValue(Task.TITLE));
-
         synchronized(controls) {
             for(TaskEditControlSet controlSet : controls)
                 controlSet.readFromTask(model);
@@ -849,6 +829,12 @@ public final class TaskEditActivity extends Fragment {
                 Toast.LENGTH_SHORT).show();
     }
 
+    protected void commentsButtonClick() {
+        Intent launchIntent = new Intent(getActivity(), EditNoteActivity.class);
+        launchIntent.putExtra(EditNoteActivity.EXTRA_TASK_ID, model.getId());
+        startActivity(launchIntent);
+    }
+
     @Override
     public boolean onOptionsItemSelected(MenuItem item) {
         switch(item.getItemId()) {
@@ -861,6 +847,9 @@ public final class TaskEditActivity extends Fragment {
         case MENU_DELETE_ID:
             deleteButtonClick();
             return true;
+        case MENU_COMMENTS_ID:
+            commentsButtonClick();
+            return true;
         case android.R.id.home:
             saveButtonClick();
             return true;
@@ -885,6 +874,10 @@ public final class TaskEditActivity extends Fragment {
         item = menu.add(Menu.NONE, MENU_DELETE_ID, 0, R.string.TEA_menu_delete);
         item.setIcon(android.R.drawable.ic_menu_delete);
         item.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM);
+
+        item = menu.add(Menu.NONE, MENU_COMMENTS_ID, 0, R.string.TEA_menu_comments);
+        item.setIcon(R.drawable.icn_cmmt_off);
+        item.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM);
     }
 
     @Override