Code review comment for lp:tomdroid/beta

Revision history for this message
Mads Boserup Lauritsen (madsbola) wrote :

Hey Noah

Some comments on the first testing.

Fantastic that it is now possible to edit notes.

Best regards
/Mads

ps. As I mentioned I am not a coding guy, but if you ever comes to Copenhagen I'll certainly will offer you a homegrown tomato from our roofgarden. Check out www.tagtomat.dk for pictures of the process.
pps. To all the other developers who does a great job. Thx, I really appreciate it, and would as a "TomatoGuy" be more than willing to share a beer or two if you pass Copenhagen once in the future.

Sync
I just testet it on Xperia X10 mini pro, and it syncs without problem with U1 (running android 2.3.7 CynogenMod).

Editing
Works well, simply fantastic :-)

Maybe related to Lucas comment a suggestion could be. When you press the "back" button the note is saved. Now I just jumped around the different notes, and found out that you need to press "save note" after editing. Of course it works (and thx for that).

Links
I could not get links to work, neither the internal links between notes (headlines) nor external links like www.ubuntu.com

Installing
I had to uninstall the Tomdroid 0.5 manually before installing the new APK, but it installs fine and runs afterwards.

Blinking screen
The same as Stefan mentions above.

Home Screen
The top bar of the screen is pure black. At the earlier version the top bar was yellow and a small sync icon showed, but maybe it is not needed now, if the app syncs at startup.

Shortcuts to the home screen
I tried to create shortcuts of the notes. It creates the shortcut, but the specific shortcut does not open on press.

« Back to merge proposal