From c8211ea1cc65560938c31263edce6fcb87016acb Mon Sep 17 00:00:00 2001 From: Tom O'Neill Date: Thu, 22 Aug 2013 14:28:12 -0700 Subject: Add a little comment about the enabled value Change-Id: I4ac0b864f55992242b6a3b0d8ffb328f23f6b645 --- location/java/android/location/SettingInjectorService.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'location/java') diff --git a/location/java/android/location/SettingInjectorService.java b/location/java/android/location/SettingInjectorService.java index 4bfa150f1d08..dbc3f276fb2f 100644 --- a/location/java/android/location/SettingInjectorService.java +++ b/location/java/android/location/SettingInjectorService.java @@ -176,8 +176,15 @@ public abstract class SettingInjectorService extends IntentService { /** * Constructor. + *

+ * Note that to prevent churn in the settings list, there is no support for dynamically + * choosing to hide a setting. Instead you should provide a {@code enabled} value of false, + * which will gray the setting out and disable the link from "Settings > Location" + * to your setting activity. One reason why you might choose to do this is if + * {@link android.provider.Settings.Secure#getLocationMode(android.content.ContentResolver)} + * is {@link android.provider.Settings.Secure#LOCATION_MODE_OFF}. * - * @param summary the {@link Preference#getSummary()} value + * @param summary the {@link Preference#getSummary()} value (allowed to be null or empty) * @param enabled the {@link Preference#isEnabled()} value */ public Status(String summary, boolean enabled) { -- cgit v1.2.3-59-g8ed1b