Merge lp:~rexbron/astrid/num_keyboard_fix into lp:~tim-todoroo/astrid/2.8.7

Proposed by Andrew Hunter
Status: Merged
Merged at revision: not available
Proposed branch: lp:~rexbron/astrid/num_keyboard_fix
Merge into: lp:~tim-todoroo/astrid/2.8.7
Diff against target: None lines
To merge this branch: bzr merge lp:~rexbron/astrid/num_keyboard_fix
Reviewer Review Type Date Requested Status
Tim Su Pending
Review via email: mp+8825@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andrew Hunter (rexbron) wrote :

Also suggesting to merge this into the 2.8.7 bugfix release.

This is the first piece of Android software I've hacked on. I was able to test it in an emulator but don't have spare hardware to do an on device test. Hopefully I haven't made any mistakes.

Revision history for this message
Tim Su (tim-todoroo) wrote :

> Also suggesting to merge this into the 2.8.7 bugfix release.
>
> This is the first piece of Android software I've hacked on. I was able to test
> it in an emulator but don't have spare hardware to do an on device test.
> Hopefully I haven't made any mistakes.

Looks good. I'm not going to incorporate the manifest and default.properties file, but otherwise, I'm going to go ahead with the merge. Thanks so much!

Revision history for this message
Andrew Hunter (rexbron) wrote :

> > Also suggesting to merge this into the 2.8.7 bugfix release.
> >
> > This is the first piece of Android software I've hacked on. I was able to
> test
> > it in an emulator but don't have spare hardware to do an on device test.
> > Hopefully I haven't made any mistakes.
>
> Looks good. I'm not going to incorporate the manifest and default.properties
> file, but otherwise, I'm going to go ahead with the merge. Thanks so much!

Cool, I updated the project to get the build.xml file to try and build astrid from the command line but ended up using eclipse instead.

Revision history for this message
Tim Su (tim-todoroo) wrote :

Awesome. Last thing: you should mark this branch as "merged" instead of
"development", since you're done with it.

Did you know that if you have an Android phone, you can do development
on your phone? The only caveat is if you already have Astrid installed
from the market, you'll have to uninstall, which means you may want to
back up your data (i.e. from /data/data/com.timsu.astrid). Let me know
if you want more info about this.

Thanks again!

Andrew Hunter wrote:
>>> Also suggesting to merge this into the 2.8.7 bugfix release.
>>>
>>> This is the first piece of Android software I've hacked on. I was able to
>>>
>> test
>>
>>> it in an emulator but don't have spare hardware to do an on device test.
>>> Hopefully I haven't made any mistakes.
>>>
>> Looks good. I'm not going to incorporate the manifest and default.properties
>> file, but otherwise, I'm going to go ahead with the merge. Thanks so much!
>>
>
> Cool, I updated the project to get the build.xml file to try and build astrid from the command line but ended up using eclipse instead.
>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'AndroidManifest.xml'
--- AndroidManifest.xml 2009-06-13 20:37:20 +0000
+++ AndroidManifest.xml 2009-07-14 16:14:39 +0000
@@ -16,7 +16,7 @@
16 <uses-permission android:name="android.permission.READ_PHONE_STATE" />16 <uses-permission android:name="android.permission.READ_PHONE_STATE" />
17 <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />17 <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
18 18
19 <uses-sdk android:minSdkVersion="1" />19 <uses-sdk android:minSdkVersion="3"/>
20 20
21 <application android:icon="@drawable/icon" android:label="@string/app_name" android:debuggable="true">21 <application android:icon="@drawable/icon" android:label="@string/app_name" android:debuggable="true">
22 22
@@ -97,4 +97,5 @@
9797
98 </application> 98 </application>
99 99
100
100</manifest> 101</manifest>
101102
=== modified file 'default.properties'
--- default.properties 2009-05-18 00:37:17 +0000
+++ default.properties 2009-07-14 16:14:39 +0000
@@ -7,8 +7,6 @@
7# "build.properties", and override values to adapt the script to your7# "build.properties", and override values to adapt the script to your
8# project structure.8# project structure.
99
10# Project target.
11target=android-2
12# apk configurations. This property allows creation of APK files with limited10# apk configurations. This property allows creation of APK files with limited
13# resources. For example, if your application contains many locales and11# resources. For example, if your application contains many locales and
14# you wish to release multiple smaller apks instead of a large one, you can12# you wish to release multiple smaller apks instead of a large one, you can
@@ -20,3 +18,5 @@
20# apk-config-european=en,fr,it,de,es18# apk-config-european=en,fr,it,de,es
21# apk-config-northamerica=en,es19# apk-config-northamerica=en,es
22apk-configurations=20apk-configurations=
21# Project target.
22target=android-3
2323
=== modified file 'res/layout/number_picker.xml'
--- res/layout/number_picker.xml 2008-12-26 10:22:13 +0000
+++ res/layout/number_picker.xml 2009-07-14 16:14:39 +0000
@@ -28,7 +28,7 @@
28 android:layout_width="fill_parent"28 android:layout_width="fill_parent"
29 android:layout_height="wrap_content"29 android:layout_height="wrap_content"
30 android:gravity="center"30 android:gravity="center"
31 android:singleLine="true"31 android:inputType="time"
32 style="?android:attr/textAppearanceLargeInverse"32 style="?android:attr/textAppearanceLargeInverse"
33 android:textSize="30sp"33 android:textSize="30sp"
34 android:background="@drawable/timepicker_input"34 android:background="@drawable/timepicker_input"
@@ -40,4 +40,4 @@
40 android:background="@drawable/timepicker_down_btn"40 android:background="@drawable/timepicker_down_btn"
41 />41 />
42 42
43</merge>
44\ No newline at end of file43\ No newline at end of file
44</merge>
4545
=== modified file 'res/layout/number_picker_dialog.xml'
--- res/layout/number_picker_dialog.xml 2008-12-26 10:22:13 +0000
+++ res/layout/number_picker_dialog.xml 2009-07-14 16:14:39 +0000
@@ -23,4 +23,4 @@
23 <com.timsu.astrid.widget.NumberPicker android:id="@+id/numberPicker"23 <com.timsu.astrid.widget.NumberPicker android:id="@+id/numberPicker"
24 android:layout_width="fill_parent"24 android:layout_width="fill_parent"
25 android:layout_height="fill_parent"/>25 android:layout_height="fill_parent"/>
26</FrameLayout>
27\ No newline at end of file26\ No newline at end of file
27</FrameLayout>

Subscribers

People subscribed via source and target branches

to all changes: