Bug 1312477 - Pre: increase touch-target size to Android UI guidelines r?sebastian draft
authorAndrzej Hunt <ahunt@mozilla.com>
Wed, 02 Nov 2016 20:58:26 +0100
changeset 432797 c9f931604d51e833be1255f4e5812beb2792d7fd
parent 432540 7247cfcf5a00155081ff10bab85c2b1922439f0f
child 432798 49ebe98a0ce759da2554834355a2abd1a9b7e796
push id34426
push userahunt@mozilla.com
push dateWed, 02 Nov 2016 20:14:04 +0000
reviewerssebastian
bugs1312477
milestone52.0a1
Bug 1312477 - Pre: increase touch-target size to Android UI guidelines r?sebastian 48dp is the minimum touch target size as suggested in the Android UI guidelines: https://developer.android.com/design/patterns/accessibility.html A similar recommendation is made in: https://material.google.com/layout/metrics-keylines.html#metrics-keylines-touch-target-size MozReview-Commit-ID: 9ybgjSOfD2J
mobile/android/base/resources/values/dimens.xml
--- a/mobile/android/base/resources/values/dimens.xml
+++ b/mobile/android/base/resources/values/dimens.xml
@@ -223,10 +223,10 @@
     <item name="activity_stream_top_sites_text_height" type="dimen">30dp</item>
 
     <item name="activity_stream_contextmenu_peek_height" type="dimen">380dp</item>
     <!-- note: max_menu_height only affects the scrolling menu, but doesnt' take into consideration
          the header above it. -->
     <item name="activity_stream_contextmenu_max_menu_height" type="dimen">350dp</item>
 
     <!-- Default touch target size for buttons/imageviews that might be of small size -->
-    <item name="touch_target_size" type="dimen">40dp</item>
+    <item name="touch_target_size" type="dimen">48dp</item>
 </resources>