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
1=== modified file 'AndroidManifest.xml'
2--- AndroidManifest.xml 2009-06-13 20:37:20 +0000
3+++ AndroidManifest.xml 2009-07-14 16:14:39 +0000
4@@ -16,7 +16,7 @@
5 <uses-permission android:name="android.permission.READ_PHONE_STATE" />
6 <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
7
8- <uses-sdk android:minSdkVersion="1" />
9+ <uses-sdk android:minSdkVersion="3"/>
10
11 <application android:icon="@drawable/icon" android:label="@string/app_name" android:debuggable="true">
12
13@@ -97,4 +97,5 @@
14
15 </application>
16
17+
18 </manifest>
19
20=== modified file 'default.properties'
21--- default.properties 2009-05-18 00:37:17 +0000
22+++ default.properties 2009-07-14 16:14:39 +0000
23@@ -7,8 +7,6 @@
24 # "build.properties", and override values to adapt the script to your
25 # project structure.
26
27-# Project target.
28-target=android-2
29 # apk configurations. This property allows creation of APK files with limited
30 # resources. For example, if your application contains many locales and
31 # you wish to release multiple smaller apks instead of a large one, you can
32@@ -20,3 +18,5 @@
33 # apk-config-european=en,fr,it,de,es
34 # apk-config-northamerica=en,es
35 apk-configurations=
36+# Project target.
37+target=android-3
38
39=== modified file 'res/layout/number_picker.xml'
40--- res/layout/number_picker.xml 2008-12-26 10:22:13 +0000
41+++ res/layout/number_picker.xml 2009-07-14 16:14:39 +0000
42@@ -28,7 +28,7 @@
43 android:layout_width="fill_parent"
44 android:layout_height="wrap_content"
45 android:gravity="center"
46- android:singleLine="true"
47+ android:inputType="time"
48 style="?android:attr/textAppearanceLargeInverse"
49 android:textSize="30sp"
50 android:background="@drawable/timepicker_input"
51@@ -40,4 +40,4 @@
52 android:background="@drawable/timepicker_down_btn"
53 />
54
55-</merge>
56\ No newline at end of file
57+</merge>
58
59=== modified file 'res/layout/number_picker_dialog.xml'
60--- res/layout/number_picker_dialog.xml 2008-12-26 10:22:13 +0000
61+++ res/layout/number_picker_dialog.xml 2009-07-14 16:14:39 +0000
62@@ -23,4 +23,4 @@
63 <com.timsu.astrid.widget.NumberPicker android:id="@+id/numberPicker"
64 android:layout_width="fill_parent"
65 android:layout_height="fill_parent"/>
66-</FrameLayout>
67\ No newline at end of file
68+</FrameLayout>

Subscribers

People subscribed via source and target branches

to all changes: