From 94adb773909b34086bfef12897e0c034c5116d99 Mon Sep 17 00:00:00 2001 From: Tai Kuo Date: Fri, 12 Jun 2020 21:47:32 +0800 Subject: walt: app: fix Lint errors in AndroidManifest & resource files Fix Lint errors in resource files and AndroidManifest.xml Bug: 149721303 Test: ./gradlew build --stacktrace --warning-mode all Signed-off-by: Tai Kuo Change-Id: I2c2c84370c37da1ee07fa4cfa9880554d0f96242 --- android/WALT/app/src/main/AndroidManifest.xml | 6 ++- .../main/res/drawable/ic_music_note_black_24dp.xml | 2 +- .../main/res/drawable/ic_settings_black_24dp.xml | 2 +- .../drawable/ic_system_update_alt_black_24dp.xml | 2 +- .../WALT/app/src/main/res/layout/activity_main.xml | 29 +++++------- .../WALT/app/src/main/res/layout/dialog_upload.xml | 3 +- .../src/main/res/layout/fragment_accelerometer.xml | 9 ++-- .../app/src/main/res/layout/fragment_audio.xml | 14 +++--- .../src/main/res/layout/fragment_diagnostics.xml | 33 ++++++++------ .../src/main/res/layout/fragment_drag_latency.xml | 17 ++++--- .../src/main/res/layout/fragment_front_page.xml | 52 +++++++++++++--------- .../WALT/app/src/main/res/layout/fragment_midi.xml | 6 ++- .../main/res/layout/fragment_screen_response.xml | 14 +++--- .../src/main/res/layout/fragment_tap_latency.xml | 12 ++--- android/WALT/app/src/main/res/layout/histogram.xml | 6 +-- .../WALT/app/src/main/res/layout/line_chart.xml | 6 +-- android/WALT/app/src/main/res/layout/toolbar.xml | 1 - android/WALT/app/src/main/res/menu/menu_main.xml | 6 +-- android/WALT/app/src/main/res/values/dimens.xml | 1 - android/WALT/app/src/main/res/values/strings.xml | 32 ++++++++++++- android/WALT/app/src/main/res/values/styles.xml | 1 - android/WALT/app/src/main/res/xml/backup_rules.xml | 4 ++ 22 files changed, 157 insertions(+), 101 deletions(-) create mode 100644 android/WALT/app/src/main/res/xml/backup_rules.xml (limited to 'android') diff --git a/android/WALT/app/src/main/AndroidManifest.xml b/android/WALT/app/src/main/AndroidManifest.xml index 869b5e8..992d368 100644 --- a/android/WALT/app/src/main/AndroidManifest.xml +++ b/android/WALT/app/src/main/AndroidManifest.xml @@ -1,5 +1,6 @@ @@ -9,9 +10,12 @@ + android:theme="@style/AppTheme" + tools:ignore="LockedOrientationActivity"> + android:pathData="M12 3v10.55c-0.59-0.34-1.27-0.55-2-0.55-2.21 0-4 1.79-4 4s1.79 4 4 4 4-1.79 4-4V7h4V3h-6z"/> \ No newline at end of file diff --git a/android/WALT/app/src/main/res/drawable/ic_settings_black_24dp.xml b/android/WALT/app/src/main/res/drawable/ic_settings_black_24dp.xml index ace746c..896ea9d 100644 --- a/android/WALT/app/src/main/res/drawable/ic_settings_black_24dp.xml +++ b/android/WALT/app/src/main/res/drawable/ic_settings_black_24dp.xml @@ -5,5 +5,5 @@ android:viewportHeight="24.0"> + android:pathData="M19.1,12.9a2.8,2.8 0,0 0,0.1 -0.9,2.8 2.8,0 0,0 -0.1,-0.9l2.1,-1.6a0.7,0.7 0,0 0,0.1 -0.6L19.4,5.5a0.7,0.7 0,0 0,-0.6 -0.2l-2.4,1a6.5,6.5 0,0 0,-1.6 -0.9l-0.4,-2.6a0.5,0.5 0,0 0,-0.5 -0.4H10.1a0.5,0.5 0,0 0,-0.5 0.4L9.3,5.4a5.6,5.6 0,0 0,-1.7 0.9l-2.4,-1a0.4,0.4 0,0 0,-0.5 0.2l-2,3.4c-0.1,0.2 0,0.4 0.2,0.6l2,1.6a2.8,2.8 0,0 0,-0.1 0.9,2.8 2.8,0 0,0 0.1,0.9L2.8,14.5a0.7,0.7 0,0 0,-0.1 0.6l1.9,3.4a0.7,0.7 0,0 0,0.6 0.2l2.4,-1a6.5,6.5 0,0 0,1.6 0.9l0.4,2.6a0.5,0.5 0,0 0,0.5 0.4h3.8a0.5,0.5 0,0 0,0.5 -0.4l0.3,-2.6a5.6,5.6 0,0 0,1.7 -0.9l2.4,1a0.4,0.4 0,0 0,0.5 -0.2l2,-3.4c0.1,-0.2 0,-0.4 -0.2,-0.6ZM12,15.6A3.6,3.6 0,1 1,15.6 12,3.6 3.6,0 0,1 12,15.6Z"/> diff --git a/android/WALT/app/src/main/res/drawable/ic_system_update_alt_black_24dp.xml b/android/WALT/app/src/main/res/drawable/ic_system_update_alt_black_24dp.xml index ec96a71..6ce20c7 100644 --- a/android/WALT/app/src/main/res/drawable/ic_system_update_alt_black_24dp.xml +++ b/android/WALT/app/src/main/res/drawable/ic_system_update_alt_black_24dp.xml @@ -5,5 +5,5 @@ android:viewportHeight="24.0"> + android:pathData="M12 16.5l4-4h-3v-9h-2v9H8l4 4zm9-13h-6v1.99h6v14.03H3V5.49h6V3.5H3c-1.1 0-2 0.9-2 2v14c0 1.1 0.9 2 2 2h18c1.1 0 2-0.9 2-2v-14c0-1.1-0.9-2-2-2z"/> diff --git a/android/WALT/app/src/main/res/layout/activity_main.xml b/android/WALT/app/src/main/res/layout/activity_main.xml index 57e7679..ad9a864 100644 --- a/android/WALT/app/src/main/res/layout/activity_main.xml +++ b/android/WALT/app/src/main/res/layout/activity_main.xml @@ -1,27 +1,20 @@ - - - - - - - + + - - + - + \ No newline at end of file diff --git a/android/WALT/app/src/main/res/layout/dialog_upload.xml b/android/WALT/app/src/main/res/layout/dialog_upload.xml index be990a2..88983d7 100644 --- a/android/WALT/app/src/main/res/layout/dialog_upload.xml +++ b/android/WALT/app/src/main/res/layout/dialog_upload.xml @@ -9,7 +9,8 @@ android:id="@+id/edit_text" android:layout_width="match_parent" android:layout_height="wrap_content" - android:hint="Enter URL" + android:hint="@string/hint_url" + android:autofillHints="Enter URL" android:inputType="textUri" /> + app:srcCompat="@drawable/ic_play_arrow_black_24dp" + android:contentDescription="@string/start" /> @@ -56,11 +57,11 @@ android:id="@+id/button_close_chart" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_alignParentRight="true" - android:layout_alignParentTop="true" + android:layout_alignTop="@id/latency_chart" + android:layout_alignEnd="@id/latency_chart" android:tint="@color/button_tint" android:layout_margin="5dp" - android:text="Close" /> + android:text="@string/close" /> + walt:srcCompat="@drawable/ic_stop_black_24dp" + android:contentDescription="@string/stop" /> + walt:srcCompat="@drawable/ic_play_arrow_black_24dp" + android:contentDescription="@string/start" /> diff --git a/android/WALT/app/src/main/res/layout/fragment_diagnostics.xml b/android/WALT/app/src/main/res/layout/fragment_diagnostics.xml index 262f616..2950bac 100644 --- a/android/WALT/app/src/main/res/layout/fragment_diagnostics.xml +++ b/android/WALT/app/src/main/res/layout/fragment_diagnostics.xml @@ -24,7 +24,8 @@ + app:srcCompat="@drawable/ic_usb_black_24dp" + android:contentDescription="@string/reconnect" /> + android:text="@string/reconnect" /> + android:text="@string/tbd_conn_status" /> @@ -55,7 +56,8 @@ + app:srcCompat="@drawable/ic_swap_horiz_black_24dp" + android:contentDescription="@string/ping" /> + android:text="@string/ping" /> + android:text="@string/ping_over_usb_with_1_byte" /> @@ -85,7 +87,8 @@ + app:srcCompat="@drawable/ic_schedule_black_24dp" + android:contentDescription="@string/re_sync_clocks" /> + android:text="@string/re_sync_clocks" /> @@ -110,7 +113,8 @@ + app:srcCompat="@drawable/ic_timelapse_black_24dp" + android:contentDescription="@string/check_clock_drift" /> + android:text="@string/check_clock_drift" /> + android:text="@string/check_how_much_clocks_diverged" /> @@ -140,7 +144,8 @@ + app:srcCompat="@drawable/ic_system_update_alt_black_24dp" + android:contentDescription="@string/update_walt_firmware" /> + android:text="@string/update_walt_firmware" /> + android:text="@string/please_press_the_button_on_the_teensy_first" /> diff --git a/android/WALT/app/src/main/res/layout/fragment_drag_latency.xml b/android/WALT/app/src/main/res/layout/fragment_drag_latency.xml index 9e77ff9..a5e36c6 100644 --- a/android/WALT/app/src/main/res/layout/fragment_drag_latency.xml +++ b/android/WALT/app/src/main/res/layout/fragment_drag_latency.xml @@ -20,21 +20,24 @@ android:layout_width="wrap_content" android:layout_height="wrap_content" android:tint="@color/button_tint" - app:srcCompat="@drawable/ic_refresh_black_24dp" /> + app:srcCompat="@drawable/ic_refresh_black_24dp" + android:contentDescription="@string/restart" /> + app:srcCompat="@drawable/ic_play_arrow_black_24dp" + android:contentDescription="@string/start" /> + app:srcCompat="@drawable/ic_check_black_24dp" + android:contentDescription="@string/finish" /> + android:text="@string/close" /> + app:srcCompat="@drawable/ic_search_black_24dp" + android:contentDescription="@string/diagnostics" /> + android:text="@string/diagnostics" /> @@ -48,7 +49,8 @@ + app:srcCompat="@drawable/ic_radio_button_checked_black_24dp" + android:contentDescription="@string/tap_latency" /> + android:text="@string/tap_latency" /> @@ -73,7 +75,8 @@ + app:srcCompat="@drawable/ic_swap_vert_black_24dp" + android:contentDescription="@string/drag_latency" /> + android:text="@string/drag_latency" /> @@ -98,7 +101,8 @@ + app:srcCompat="@drawable/ic_brightness_medium_black_24dp" + android:contentDescription="@string/screen_response" /> + android:text="@string/screen_response" /> @@ -123,7 +127,8 @@ + app:srcCompat="@drawable/ic_volume_up_black_24dp" + android:contentDescription="@string/audio_latency" /> + android:text="@string/audio_latency" /> @@ -150,7 +155,8 @@ android:id="@+id/midi_image" style="@style/MenuIconStyle" android:tint="@color/ColorDisabled" - app:srcCompat="@drawable/ic_music_note_black_24dp" /> + app:srcCompat="@drawable/ic_music_note_black_24dp" + android:contentDescription="@string/midi_latency" /> + android:text="@string/midi_latency" /> @@ -177,7 +183,8 @@ + app:srcCompat="@drawable/ic_receipt_black_24dp" + android:contentDescription="@string/view_log" /> + android:text="@string/view_log" /> @@ -202,7 +209,8 @@ + app:srcCompat="@drawable/ic_swap_horiz_black_24dp" + android:contentDescription="@string/accelerometer_latency" /> + android:text="@string/accelerometer_latency" /> @@ -227,7 +235,8 @@ + app:srcCompat="@drawable/ic_settings_black_24dp" + android:contentDescription="@string/settings" /> + android:text="@string/settings" /> @@ -253,7 +262,8 @@ + app:srcCompat="@drawable/ic_help_outline_black_24dp" + android:contentDescription="@string/about" /> + android:text="@string/about" /> diff --git a/android/WALT/app/src/main/res/layout/fragment_midi.xml b/android/WALT/app/src/main/res/layout/fragment_midi.xml index 93beb58..fba287f 100644 --- a/android/WALT/app/src/main/res/layout/fragment_midi.xml +++ b/android/WALT/app/src/main/res/layout/fragment_midi.xml @@ -21,7 +21,8 @@ android:layout_height="wrap_content" android:layout_alignParentStart="true" android:tint="@color/button_tint" - walt:srcCompat="@drawable/ic_input_black_24dp" /> + walt:srcCompat="@drawable/ic_input_black_24dp" + android:contentDescription="@string/start_midi_in" /> + walt:srcCompat="@drawable/ic_output_black_24dp" + android:contentDescription="@string/start_midi_out" /> + walt:srcCompat="@drawable/ic_stop_black_24dp" + android:contentDescription="@string/stop" /> + walt:srcCompat="@drawable/ic_play_arrow_black_24dp" + android:contentDescription="@string/start" /> diff --git a/android/WALT/app/src/main/res/layout/fragment_tap_latency.xml b/android/WALT/app/src/main/res/layout/fragment_tap_latency.xml index ab2ff51..28f4327 100644 --- a/android/WALT/app/src/main/res/layout/fragment_tap_latency.xml +++ b/android/WALT/app/src/main/res/layout/fragment_tap_latency.xml @@ -18,17 +18,19 @@ android:id="@+id/button_finish_tap" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_alignParentLeft="true" + android:layout_alignParentStart="true" android:tint="@color/button_tint" - walt:srcCompat="@drawable/ic_check_black_24dp" /> + walt:srcCompat="@drawable/ic_check_black_24dp" + android:contentDescription="@string/finish" /> + walt:srcCompat="@drawable/ic_play_arrow_black_24dp" + android:contentDescription="@string/restart" /> + android:text="@string/close" /> diff --git a/android/WALT/app/src/main/res/layout/line_chart.xml b/android/WALT/app/src/main/res/layout/line_chart.xml index 7de4cfb..a846a13 100644 --- a/android/WALT/app/src/main/res/layout/line_chart.xml +++ b/android/WALT/app/src/main/res/layout/line_chart.xml @@ -15,9 +15,9 @@ android:id="@+id/button_close_chart" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_alignParentRight="true" - android:layout_alignParentTop="true" + android:layout_alignTop="@id/chart" + android:layout_alignEnd="@id/chart" android:tint="@color/button_tint" android:layout_margin="5dp" - android:text="Close" /> + android:text="@string/close" /> diff --git a/android/WALT/app/src/main/res/layout/toolbar.xml b/android/WALT/app/src/main/res/layout/toolbar.xml index ef6aea8..c932cdb 100644 --- a/android/WALT/app/src/main/res/layout/toolbar.xml +++ b/android/WALT/app/src/main/res/layout/toolbar.xml @@ -2,7 +2,6 @@ \ No newline at end of file diff --git a/android/WALT/app/src/main/res/menu/menu_main.xml b/android/WALT/app/src/main/res/menu/menu_main.xml index 6333e6d..e34957b 100644 --- a/android/WALT/app/src/main/res/menu/menu_main.xml +++ b/android/WALT/app/src/main/res/menu/menu_main.xml @@ -7,21 +7,21 @@ diff --git a/android/WALT/app/src/main/res/values/dimens.xml b/android/WALT/app/src/main/res/values/dimens.xml index 47c8224..5ee798a 100644 --- a/android/WALT/app/src/main/res/values/dimens.xml +++ b/android/WALT/app/src/main/res/values/dimens.xml @@ -1,5 +1,4 @@ 16dp - 16dp diff --git a/android/WALT/app/src/main/res/values/strings.xml b/android/WALT/app/src/main/res/values/strings.xml index 7204eaa..bc83273 100644 --- a/android/WALT/app/src/main/res/values/strings.xml +++ b/android/WALT/app/src/main/res/values/strings.xml @@ -1,7 +1,6 @@ WALT - Settings Crash Log WALT reports protocol version %1$s, which is not supported. Please program WALT to a firmware with protocol version %2$s. To do this @@ -16,6 +15,7 @@ DISCLAIMER: This is not an official Google product. A WALT device is required to run the latency tests. For more information, visit github.com/google/walt Privacy policy:\ngithub.com/google/walt/blob/master/docs/PrivacyPolicy.md + Enter URL pref_screen_blinks pref_audio_in_reps pref_audio_in_threshold @@ -29,6 +29,36 @@ pref_screen_fullscreen pref_log_url pref_auto_upload_log + Start + Close + Reconnect + TBD: Conn status + Tap latency + Drag latency + Screen response + Audio latency + MIDI latency + View log + Accelerometer latency + Settings + About + Diagnostics + TBD: Connection/sync status + Ping + Ping over USB with 1 byte + Re-sync clocks + Check clock drift + Check how much clocks diverged + Update WALT Firmware + Please press the button on the Teensy first + Finish + Restart + Stop + Start MIDI in + Start MIDI out + Upload + Share + Help Continuous Playback Latency Continuous Recording Latency diff --git a/android/WALT/app/src/main/res/values/styles.xml b/android/WALT/app/src/main/res/values/styles.xml index ac4619e..05e7a67 100644 --- a/android/WALT/app/src/main/res/values/styles.xml +++ b/android/WALT/app/src/main/res/values/styles.xml @@ -7,7 +7,6 @@ @color/ColorAccent @style/PreferenceThemeOverlay.v14.Material @style/ImageButtonStyle - @style/ImageButtonStyle diff --git a/android/WALT/app/src/main/res/xml/backup_rules.xml b/android/WALT/app/src/main/res/xml/backup_rules.xml new file mode 100644 index 0000000..0e782d2 --- /dev/null +++ b/android/WALT/app/src/main/res/xml/backup_rules.xml @@ -0,0 +1,4 @@ + + + + \ No newline at end of file -- cgit v1.2.3