From 36f4b977129fb4c4259e08af5376eb239776d2c9 Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Fri, 24 May 2019 11:19:06 -0700 Subject: [PATCH] For #658: Adds your rights page link --- .../org/mozilla/fenix/settings/SettingsFragment.kt | 14 ++++++++++++-- .../org/mozilla/fenix/settings/SupportUtils.kt | 3 ++- app/src/main/res/values/preference_keys.xml | 3 ++- app/src/main/res/values/strings.xml | 6 ++++-- app/src/main/res/xml/preferences.xml | 8 ++++++-- 5 files changed, 26 insertions(+), 8 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt index 86bece54a..38cae0269 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt @@ -37,7 +37,7 @@ import org.mozilla.fenix.R.string.pref_key_help import org.mozilla.fenix.R.string.pref_key_language import org.mozilla.fenix.R.string.pref_key_leakcanary import org.mozilla.fenix.R.string.pref_key_make_default_browser -import org.mozilla.fenix.R.string.pref_key_privacy_policy +import org.mozilla.fenix.R.string.pref_key_privacy_notice import org.mozilla.fenix.R.string.pref_key_rate import org.mozilla.fenix.R.string.pref_key_remote_debugging import org.mozilla.fenix.R.string.pref_key_search_engine_settings @@ -45,6 +45,7 @@ import org.mozilla.fenix.R.string.pref_key_sign_in import org.mozilla.fenix.R.string.pref_key_site_permissions import org.mozilla.fenix.R.string.pref_key_theme import org.mozilla.fenix.R.string.pref_key_tracking_protection_settings +import org.mozilla.fenix.R.string.pref_key_your_rights import org.mozilla.fenix.components.metrics.Event import org.mozilla.fenix.ext.components import org.mozilla.fenix.ext.getPreferenceKey @@ -163,12 +164,21 @@ class SettingsFragment : PreferenceFragmentCompat(), CoroutineScope, AccountObse resources.getString(pref_key_theme) -> { navigateToThemeSettings() } - resources.getString(pref_key_privacy_policy) -> { + resources.getString(pref_key_privacy_notice) -> { requireContext().apply { val intent = SupportUtils.createCustomTabIntent(this, SupportUtils.PRIVACY_NOTICE_URL) startActivity(intent) } } + resources.getString(pref_key_your_rights) -> { + requireContext().apply { + val intent = SupportUtils.createCustomTabIntent( + this, + SupportUtils.getSumoURLForTopic(context!!, SupportUtils.SumoTopic.YOUR_RIGHTS) + ) + startActivity(intent) + } + } } return super.onPreferenceTreeClick(preference) } diff --git a/app/src/main/java/org/mozilla/fenix/settings/SupportUtils.kt b/app/src/main/java/org/mozilla/fenix/settings/SupportUtils.kt index a847504e1..9b3b4f48f 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SupportUtils.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/SupportUtils.kt @@ -24,7 +24,8 @@ object SupportUtils { internal val topicStr: String ) { HELP("firefox-android-help"), - PRIVATE_BROWSING_MYTHS("common-myths-about-private-browsing") + PRIVATE_BROWSING_MYTHS("common-myths-about-private-browsing"), + YOUR_RIGHTS("your-rights") } fun getSumoURLForTopic(context: Context, topic: SumoTopic): String { diff --git a/app/src/main/res/values/preference_keys.xml b/app/src/main/res/values/preference_keys.xml index 684412c10..d6ced7376 100644 --- a/app/src/main/res/values/preference_keys.xml +++ b/app/src/main/res/values/preference_keys.xml @@ -17,7 +17,8 @@ pref_key_rate pref_key_feedback pref_key_about - pref_key_privacy_policy + pref_key_privacy_notice + pref_key_your_rights pref_key_account pref_key_sign_in pref_key_private_mode diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 7a5574402..8ae15d81b 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -101,8 +101,10 @@ About %1$s - - Privacy Policy + + Privacy Notice + + Your Rights Passwords diff --git a/app/src/main/res/xml/preferences.xml b/app/src/main/res/xml/preferences.xml index 5688f7fd3..84db3a55f 100644 --- a/app/src/main/res/xml/preferences.xml +++ b/app/src/main/res/xml/preferences.xml @@ -98,7 +98,11 @@ android:title="@string/preferences_about" /> + android:key="@string/pref_key_privacy_notice" + android:title="@string/preferences_privacy_notice" /> + \ No newline at end of file