Confirmed users
586
edits
No edit summary |
|||
| Line 78: | Line 78: | ||
* Setting the FilterQueryProvider automatically processes the SQL in a background thread (see http://developer.android.com/reference/android/widget/CursorAdapter.html#runQueryOnBackgroundThread%28java.lang.CharSequence%29). I verified this by adding a Thread.sleep() call in the filter - the UI remains responsive. | * Setting the FilterQueryProvider automatically processes the SQL in a background thread (see http://developer.android.com/reference/android/widget/CursorAdapter.html#runQueryOnBackgroundThread%28java.lang.CharSequence%29). I verified this by adding a Thread.sleep() call in the filter - the UI remains responsive. | ||
* Better to use styles in layout. This helps in reusability. (sriram) | * Better to use styles in layout. This helps in reusability. (sriram) | ||
* GeckoApp activity uses a lot of static variables. This has issues while rotation (they tend to hold previous values). It's better to avoid static variables and use getter and setters. [see http://stackoverflow.com/questions/2475978/using-static-variables-in-android] ( | * <strike>GeckoApp activity uses a lot of static variables. This has issues while rotation (they tend to hold previous values). It's better to avoid static variables and use getter and setters. [see http://stackoverflow.com/questions/2475978/using-static-variables-in-android]</strike> (done as of 0c46f65505f0) | ||
* Change directory structure under embedding/android to follow a more traditional Android app layout (src, res, gen, etc) for better integration with Eclipse. | * Change directory structure under embedding/android to follow a more traditional Android app layout (src, res, gen, etc) for better integration with Eclipse. | ||
* Take into account best-practices and coding conventions from http://developer.android.com/guide/practices/design/performance.html since the whole point of this is to speed up the UI. In particular, the current code: | * Take into account best-practices and coding conventions from http://developer.android.com/guide/practices/design/performance.html since the whole point of this is to speed up the UI. In particular, the current code: | ||