diff --git a/MyTracks/proguard.cfg b/MyTracks/proguard.cfg index 8a6a82b85..0be4447b2 100644 --- a/MyTracks/proguard.cfg +++ b/MyTracks/proguard.cfg @@ -35,15 +35,15 @@ -keep public class * extends android.preference.Preference -keep public class com.android.vending.licensing.ILicensingService --keepclasseswithmembernames class * { +-keepclasseswithmembers class * { native ; } --keepclasseswithmembernames class * { +-keepclasseswithmembers class * { public (android.content.Context, android.util.AttributeSet); } --keepclasseswithmembernames class * { +-keepclasseswithmembers class * { public (android.content.Context, android.util.AttributeSet, int); } diff --git a/MyTracks/res/layout/about.xml b/MyTracks/res/layout/about.xml index 37cd65890..94b7402fb 100644 --- a/MyTracks/res/layout/about.xml +++ b/MyTracks/res/layout/about.xml @@ -30,6 +30,7 @@ limitations under the License. android:layout_width="fill_parent" android:orientation="horizontal"> - - - + - - - - - - + - - - - - - + android:text="@string/chart_settings_y_axis" + style="@style/ChartSettingsAxisText"/> + android:layout_height="fill_parent"> + android:padding="10dp"> diff --git a/MyTracks/res/layout/mytracks_layout.xml b/MyTracks/res/layout/mytracks_layout.xml index 5055ea2a5..203d0636a 100644 --- a/MyTracks/res/layout/mytracks_layout.xml +++ b/MyTracks/res/layout/mytracks_layout.xml @@ -33,32 +33,30 @@ limitations under the License. android:layout_width="fill_parent" android:layout_height="wrap_content" android:background="@drawable/list_bar" - android:layout_weight="0" - android:padding="3dip" + android:padding="3dp" android:visibility="gone"> + android:layout_weight="1"/> + android:paddingTop="10dp"> + android:padding="3dp"> + android:layout_weight="1"/> + android:paddingBottom="2dp" + android:paddingTop="2dp"> @@ -53,21 +53,21 @@ limitations under the License. diff --git a/MyTracks/res/layout/mytracks_marker_item.xml b/MyTracks/res/layout/mytracks_marker_item.xml index 04130b7a6..178a58ea5 100644 --- a/MyTracks/res/layout/mytracks_marker_item.xml +++ b/MyTracks/res/layout/mytracks_marker_item.xml @@ -15,54 +15,47 @@ See the License for the specific language governing permissions and limitations under the License. --> - + android:orientation="horizontal" + android:paddingBottom="2dp" + android:paddingTop="2dp" > + - - + - - - - - - - - - + android:singleLine="true" + android:textAppearance="?android:attr/textAppearanceLarge" /> + + - + \ No newline at end of file diff --git a/MyTracks/res/layout/mytracks_waypoint_details.xml b/MyTracks/res/layout/mytracks_waypoint_details.xml index a8e1ce5d3..ab124bc5b 100644 --- a/MyTracks/res/layout/mytracks_waypoint_details.xml +++ b/MyTracks/res/layout/mytracks_waypoint_details.xml @@ -21,7 +21,7 @@ limitations under the License. android:background="#c000"> @@ -38,12 +38,13 @@ limitations under the License. android:layout_height="wrap_content" android:orientation="horizontal"> + android:paddingLeft="4dp" + android:paddingRight="2dp"/> + android:padding="1dp"> @@ -123,8 +124,8 @@ limitations under the License. android:gravity="left" android:id="@+id/total_distance_unit_label" android:layout_column="2" - android:layout_marginBottom="-9dip" - android:layout_marginTop="-9dip" + android:layout_marginBottom="-9dp" + android:layout_marginTop="-9dp" android:text="@string/unit_mile" android:textColor="@color/stat_label_text" /> + android:padding="1dp"> - android:layout_height="3dip" + android:layout_height="3dp" android:layout_width="fill_parent" /> diff --git a/MyTracks/res/layout/track_detail.xml b/MyTracks/res/layout/track_detail.xml index 58d342071..c5e30931e 100644 --- a/MyTracks/res/layout/track_detail.xml +++ b/MyTracks/res/layout/track_detail.xml @@ -43,6 +43,7 @@ limitations under the License. android:orientation="horizontal" > + android:paddingLeft="10dp" + android:paddingRight="10dp" + android:paddingTop="5dp" + android:paddingBottom="5dp"> + android:paddingLeft="15dp"> + android:paddingBottom="2dp"> @@ -72,7 +72,7 @@ limitations under the License. #FF000000 #FF0000FF - #FF444444 - #FFffd700 #FFAAAAAA - #FF00FFFF - #70000000 #FFFF0000 #FFF0F0F0 #FF0F0F0F diff --git a/MyTracks/res/values/strings.xml b/MyTracks/res/values/strings.xml index a11609227..ec8c509b1 100644 --- a/MyTracks/res/values/strings.xml +++ b/MyTracks/res/values/strings.xml @@ -921,6 +921,10 @@ limitations under the License. Screenshot: marker_detail.png"> Marker details + + Marker + @@ -975,10 +979,6 @@ limitations under the License. Screenshot: marker_list_context_menu.png"> Edit - - Marker - Track name + + Track + Show on map - - Track - gone - - - - - - @@ -152,15 +138,15 @@ limitations under the License. @@ -185,9 +171,9 @@ limitations under the License. @@ -195,8 +181,8 @@ limitations under the License. @@ -204,8 +190,8 @@ limitations under the License. diff --git a/MyTracks/src/com/google/android/apps/mytracks/io/fusiontables/SendFusionTablesAsyncTask.java b/MyTracks/src/com/google/android/apps/mytracks/io/fusiontables/SendFusionTablesAsyncTask.java index b8600dcdd..6f18ea71e 100644 --- a/MyTracks/src/com/google/android/apps/mytracks/io/fusiontables/SendFusionTablesAsyncTask.java +++ b/MyTracks/src/com/google/android/apps/mytracks/io/fusiontables/SendFusionTablesAsyncTask.java @@ -425,18 +425,26 @@ public class SendFusionTablesAsyncTask extends AbstractSendAsyncTask { } boolean isSuccess = response.isSuccessStatusCode(); if (isSuccess) { - InputStream content; + InputStream content = null; try { content = response.getContent(); + if (setTableId) { + tableId = SendFusionTablesUtils.getTableId(content); + if (tableId == null) { + Log.d(TAG, "tableId is null"); + return false; + } + } } catch (IOException e) { Log.d(TAG, "Unable to get response", e); return false; - } - if (setTableId) { - tableId = SendFusionTablesUtils.getTableId(content); - if (tableId == null) { - Log.d(TAG, "tableId is null"); - return false; + } finally { + if (content != null) { + try { + content.close(); + } catch (IOException e) { + Log.d(TAG, "Unable to close content", e); + } } } } else { diff --git a/MyTracks/src/com/google/android/apps/mytracks/io/gdata/maps/MapsGDataConverter.java b/MyTracks/src/com/google/android/apps/mytracks/io/gdata/maps/MapsGDataConverter.java index 0d22af712..fdbfa87c2 100644 --- a/MyTracks/src/com/google/android/apps/mytracks/io/gdata/maps/MapsGDataConverter.java +++ b/MyTracks/src/com/google/android/apps/mytracks/io/gdata/maps/MapsGDataConverter.java @@ -169,7 +169,6 @@ public class MapsGDataConverter { xmlSerializer.endTag(null, "Placemark"); xmlSerializer.flush(); entry.setContent(writer.toString()); - Log.d("My Google Maps", "Generated kml:\n" + entry.getContent()); Log.d("My Google Maps", "Edit URI: " + entry.getEditUri()); } catch (IOException e) { e.printStackTrace(); diff --git a/MyTracks/src/com/google/android/apps/mytracks/io/maps/ChooseMapAsyncTask.java b/MyTracks/src/com/google/android/apps/mytracks/io/maps/ChooseMapAsyncTask.java index 768de75e5..621448099 100644 --- a/MyTracks/src/com/google/android/apps/mytracks/io/maps/ChooseMapAsyncTask.java +++ b/MyTracks/src/com/google/android/apps/mytracks/io/maps/ChooseMapAsyncTask.java @@ -118,17 +118,17 @@ public class ChooseMapAsyncTask extends AsyncTask { @Override protected Boolean doInBackground(Void... params) { - return getMaps(); - } - - @Override - protected void onCancelled() { - closeClient(); + try { + return getMaps(); + } finally { + if (gDataClient != null) { + gDataClient.close(); + } + } } @Override protected void onPostExecute(Boolean result) { - closeClient(); success = result; completed = true; if (activity != null) { @@ -136,15 +136,6 @@ public class ChooseMapAsyncTask extends AsyncTask { } } - /** - * Closes the gdata client. - */ - private void closeClient() { - if (gDataClient != null) { - gDataClient.close(); - } - } - /** * Gets all the maps from Google Maps. * diff --git a/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/AbstractSendAsyncTask.java b/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/AbstractSendAsyncTask.java index 3959c5b86..710d21c6d 100644 --- a/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/AbstractSendAsyncTask.java +++ b/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/AbstractSendAsyncTask.java @@ -76,7 +76,14 @@ public abstract class AbstractSendAsyncTask extends AsyncTask