Commit 4457bb8e authored Mar 12, 2018 by Antony Sargent

Update comment for HAPTIC_FEEDBACK_ENABLED setting

The current comment is a little confusing, and this ends up in the
public documentation here:

https://developer.android.com/reference/android/provider/Settings.System.html#HAPTIC_FEEDBACK_ENABLED

See b/22390263 entry for a link to at least one instance where an actual
developer was confused by this.

This CL changes the comment to match what we call the setting in the UI
of the Settings app. Note that it's possible this could become stale
someday (e.g. we used to label this "Vibrate on touch" and switched to
"Vibrate on tap" at some point), but this description will probably
still be ok and remain an improvement over what we have now.

Bug: 22390263
Test: N/A (comment only change)
Change-Id: I717a7c5a2f9ecc38cfe6f0c1c0379a868f810782

parent fd36a5dd

core/java/android/provider/Settings.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -3727,7 +3727,7 @@ public final class Settings {
		public static final Validator SOUND_EFFECTS_ENABLED_VALIDATOR = BOOLEAN_VALIDATOR;

		/**
		* Whether the haptic feedback (long presses, ...) are enabled. The value is
		* Whether haptic feedback (Vibrate on tap) is enabled. The value is
		* boolean (1 or 0).
		*/
		public static final String HAPTIC_FEEDBACK_ENABLED = "haptic_feedback_enabled";