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 434812 5289c919ea2ac1f84084cf2a8dfb7a29a6ae21ce
parent 434811 b99fc5c3f0be858eecef9ace1a6091173962a09e
child 434813 c1c5b7111fdbcb91453fc3926e1f4d74168e31ec
push id34834
push userahunt@mozilla.com
push dateMon, 07 Nov 2016 12:22:00 +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>